aaron.ballman added subscribers: ABataev, clang-vendors.
aaron.ballman added a comment.

The patch seems large and scary, but that's because of how often we've had to 
test VLA behavior in C++ code. The OpenMP test cases are mostly opting out of 
VLA warnings mechanically (I verified with @ABataev that this was the desired 
approach) while the non-OpenMP tests were largely updated by hand so that I 
could spot check the diagnostic behavior to ensure it's reasonable. The 
functional changes themselves are quite small.

Adding the clang-vendors group in case this is disruptive for downstreams.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156565

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

Reply via email to