diff options
author | Chandler Carruth <chandlerc@gmail.com> | 2025-01-22 00:41:27 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-01-22 00:41:27 -0800 |
commit | bc6f84a2db6e7d60d70cf9be8d6cce2a101d0faa (patch) | |
tree | 19184fc6f2962a8ccfc2ce4d2900610d0370b3d4 /lldb/source/Plugins/Process/scripted/ScriptedProcess.cpp | |
parent | 0369ac8d2555423fc6c41b2aed12aadf3eb9b0cd (diff) | |
download | llvm-bc6f84a2db6e7d60d70cf9be8d6cce2a101d0faa.zip llvm-bc6f84a2db6e7d60d70cf9be8d6cce2a101d0faa.tar.gz llvm-bc6f84a2db6e7d60d70cf9be8d6cce2a101d0faa.tar.bz2 |
[StrTable] Switch diag group names to `llvm::StringTable` (#123302)
Previously, they used a hand-rolled Pascal-string encoding different
from all the other string tables produced from TableGen. This moves them
to use the newly introduced runtime abstraction, and enhances that
abstraction to support iterating over the string table as used in this
case.
From what I can tell the Pascal-string encoding isn't critical here to
avoid expensive `strlen` calls, so I think this is a simpler and more
consistent model. But if folks would prefer a Pascal-string style
encoding, I can instead work to switch the `StringTable` abstraction
towards that. It would require some tricky tradeoffs though to make it
reasonably general: either using 4 bytes instead of 1 byte to encode the
size, or having a fallback to `strlen` for long strings.
Diffstat (limited to 'lldb/source/Plugins/Process/scripted/ScriptedProcess.cpp')
0 files changed, 0 insertions, 0 deletions