diff options
| author | Alex Langford <alangford@apple.com> | 2023-06-06 18:10:48 -0700 |
|---|---|---|
| committer | Alex Langford <alangford@apple.com> | 2023-06-08 12:19:14 -0700 |
| commit | 4bae706682d5b621884ff2f7014ba4eb8cb636aa (patch) | |
| tree | b378f1ac9fc34aa5677cacafc80cd03815cb6ee8 /llvm/lib/Transforms/Utils/InlineFunction.cpp | |
| parent | 5df492302e260fa2ce8e2b7f28cf5a48902b7f8a (diff) | |
| download | llvm-4bae706682d5b621884ff2f7014ba4eb8cb636aa.zip llvm-4bae706682d5b621884ff2f7014ba4eb8cb636aa.tar.gz llvm-4bae706682d5b621884ff2f7014ba4eb8cb636aa.tar.bz2 | |
[lldb][NFCI] DecodedThread::TraceItemStorage::error should own its own data
The way it works now, it stores a `const char *` that it does not
explicitly own. It's owned by the ConstString StringPool. This is purely
to manage its lifetime, we don't really benefit from deduplication (nor
should we try to, they are errors). We also don't really benefit from
quick comparisons.
This may make the size of TraceItemStorage larger, but you have to pay
the cost of owning the data somewhere. The ConstString StringPool is an
attractive choice but ultimately a poor one.
Differential Revision: https://reviews.llvm.org/D152326
Diffstat (limited to 'llvm/lib/Transforms/Utils/InlineFunction.cpp')
0 files changed, 0 insertions, 0 deletions