[clang] [lld] [lldb] [llvm] [Option] Replace the OptionTables object with a function. NFC (PR #224805)

Alexis Engelke via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 18 23:03:50 PDT 2026


================
@@ -159,7 +159,6 @@ class LLVM_ABI OptTable {
   struct Tables {
     const StringTable &StrTable;
----------------
aengelke wrote:

Unrelated: shouldn't tthis be `StringTable StrTable`? The reference unnecessarily forces the StringTable = StringRef into memory, creating an avoidable indirection..

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


More information about the cfe-commits mailing list