arichardson added inline comments.

================
Comment at: clang/test/SemaCXX/constexpr-array-init.cpp:3
+
+
+/// expected-no-diagnostics
----------------
Nit: It might be helpful to add a comment to this test explaining what it's 
testing. This makes it easier to diagnose tests failures (e.g. in downstream 
forks) without having to rely on git blame.


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

https://reviews.llvm.org/D131155

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

Reply via email to