[Lldb-commits] [clang] [lldb] [llvm] [StrTable] Switch the option parser to `llvm::StringTable` (PR #123308)

Reid Kleckner via lldb-commits lldb-commits at lists.llvm.org
Wed Jan 22 12:47:43 PST 2025


================
@@ -117,13 +121,13 @@ class OptTable {
 private:
   // A unified string table for these options. Individual strings are stored as
   // null terminated C-strings at offsets within this table.
-  const char *StrTable;
+  const StringTable *StrTable;
----------------
rnk wrote:

Can this be a const reference instead? Is there a reason you chose to make this a pointer?

https://github.com/llvm/llvm-project/pull/123308


More information about the lldb-commits mailing list