[PATCH] D80884: [ASTMatchers] Force c++ unittests to specify correct language standard

[Nathan James via Phabricator via cfe-commits]

njames93 added a comment.

In D80884#2065270 <https://reviews.llvm.org/D80884#2065270>, @gribozavr2 wrote:

> This is an interesting question. Since Clang accepts certain constructs in older C++ standard versions as extensions, shouldn't we test AST matchers in those modes as well? I think it could be argued either way.


Clang accepts a lot of stuff for better or worse, my fist try at this was using `-pedantic-errors`, but that was throwing errors with gnu and microsoft extensions which would be a much more invasive fix to sort out.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80884