hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.

thanks, looks good to me. I'll commit for you. @gribozavr do you want to take 
another look on the patch?



================
Comment at: clang/unittests/ASTMatchers/ASTMatchersTest.h:153
+  case LanguageMode::Cxx2aOrLater:
+    LangModes = {LanguageMode::Cxx2a};
+  }
----------------
hokein wrote:
> nit: add a llvm_unreachable for `default`.
this is not done?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63149



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to