aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM aside from a minor issue.
================ Comment at: clang-tools-extra/clang-tidy/cert/DefaultOperatorNewAlignmentCheck.cpp:30-31 + // Check not applicable in C++17 or newer. + if (getLangOpts().CPlusPlus17 || getLangOpts().CPlusPlus2a) + return; + ---------------- This should be in the `registerMatchers()` function so that we don't even bother trying to match anything. You don't need to do the `||` either; if 2a is enabled, that automatically enables all previous modes as well. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67545/new/ https://reviews.llvm.org/D67545 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
