================
@@ -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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to