[PATCH] D62125: Run ClangTidy tests in all C++ language modes

Mon May 20 02:22:11 PDT 2019

gribozavr added a comment.

> c++98 coverage is a rather niche thing there days. I believe, someone with genuine need for C++98 support can work on that.

+1, that's exactly the reason why I skipped 98.  C++98 is 21 years old now (or 16 if you count from C++03).  Most projects that build with a modern toolchain and care about code quality enough to run static analysis tools have migrated to C++11.  Many ClangTidy checkers have not even been tested *ever* on 98, many (`modernize-*`) don't apply in 98.

We wouldn't refuse patches to annotate tests with C++98 support or fix bugs in C++98 mode, but I believe as far as ClangTidy is concerned C++98 is a niche use case, so I personally find it difficult to justify spending my time on it.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62125/new/

https://reviews.llvm.org/D62125