njames93 planned changes to this revision.
njames93 added a comment.
Made another pull request that could neaten this implementation up -
https://reviews.llvm.org/D75441, will wait to see how that goes down before
getting this pushed.
================
Comment at:
clang-tools-extra/clang-tidy/cert/DefaultOperatorNewAlignmentCheck.h:30
+ bool isLanguageVersionSupported(const LangOptions &LangOpts) const override {
+ return LangOpts.CPlusPlus && !LangOpts.CPlusPlus17;
+ }
----------------
gribozavr2 wrote:
> I think CPlusPlus17 implies CPlusPlus.
This is saying it needs c++ but not c++17, so c++98->c++14 are Ok but not 17 or
20. Besides this is the current behaviour in the cpp file
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75340/new/
https://reviews.llvm.org/D75340
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
