[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