owenca wrote: > @owenca Are you against converting all these `AllowShort*` bool options to > enums and having four options for each (`Never`, `EmptyAndAttached`, `Empty`, > `Always`)?
It's up to @HazardyKnusperkeks and @mydeveloperday, but IMO small, incremental patches are easier to review. https://github.com/llvm/llvm-project/pull/196021 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
