aaron.ballman added a comment. In D111228#3051686 <https://reviews.llvm.org/D111228#3051686>, @balazske wrote:
> Probably it can work to only have a "add" or "remove" possibility in the > config string, and not using an option name like in the provided example. > Otherwise it can be possible to mix string lists from different checks, that > seems to be a difficult problem. Other solution is to have a special list (or > set) configuration format. Hmm, I was thinking that was more of a feature than a problem. We have some checks that already have parallel lists of somewhat related strings, so being able to say "define this list in terms of that list plus/minus these elements" seems useful. Or are you saying there's an implementation difficulty with supporting that which might make it less of a good approach? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111228/new/ https://reviews.llvm.org/D111228 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
