[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