================
@@ -169,6 +198,10 @@ class LLVM_ABI OptTable {
   OptTable(const StringTable &StrTable,
            ArrayRef<StringTable::Offset> PrefixesTable,
            ArrayRef<Info> OptionInfos, bool IgnoreCase = false);
----------------
PiJoules wrote:

Could probably just add 

```
ArrayRef<Command> Commands = {},
ArrayRef<unsigned> CommandIDsTable = {},
```

so you don't need to make another ctor. Same with other OpTables below.

https://github.com/llvm/llvm-project/pull/155026
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to