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

Miklos Vajna via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun May 7 13:23:16 PDT 2017


vmiklos added a comment.

In https://reviews.llvm.org/D32945#748225, @malcolm.parsons wrote:

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


Ah, I see. Then yes, I think it makes sense; e.g. for the above cppunit case ideally I want to avoid any kind of warning from macros, since none of them can be easily fixed in system headers. (And the other way around, if system headers are not a problem, then all kind of warnings inside macros are probably interesting.) The updated patch uses `getLocalOrGlobal()` for the new option.


https://reviews.llvm.org/D32945





More information about the cfe-commits mailing list