MaskRay wrote:

> > Derive the prefix union from the prefix table in the constructor and drop 
> > the emitted OptionPrefixesUnion.
> 
> What's the motivation here? This doesn't look strictly required to be in the 
> same patch, and I'd prefer that to be separate.

Extracted this simplification to a separate PR.

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