shafik added inline comments.

================
Comment at: clang/test/SemaCXX/crash-params.cpp:3
+
+template <bool>
+int foo() {
----------------
If this test really does not fit into any other test set the please rename the 
test file `GH61441.cpp` so we know it is a regression test for that github bug 
report.

Otherwise if we add a test from a bug report into an existing test file then we 
wrap it something like `namespace GH61441` so we know that test is a regression 
test for github bug 61441.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146426

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

Reply via email to