[PATCH] D32945: clang-tidy: add IgnoreMacros option to modernize-use-default-member-init

Malcolm Parsons via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun May 7 11:46:02 PDT 2017


malcolm.parsons added a comment.

In https://reviews.llvm.org/D32945#748166, @vmiklos wrote:

> In https://reviews.llvm.org/D32945#748135, @malcolm.parsons wrote:
>
> > Should this option be shared?
>
>
> If the alternative is to make this a general clang-tidy option where all but these two checks ignore it, I think that's more confusing to the users. Or do you have an example option in mind that's shared only between a few checks?


See uses of `OptionsView::getLocalOrGlobal()`.


https://reviews.llvm.org/D32945





More information about the cfe-commits mailing list