owenpan accepted this revision.
owenpan added inline comments.

================
Comment at: clang/unittests/Format/FormatTest.cpp:23593-23594
-      "template <typename T>\n"
-      "concept DelayedCheck = static_cast<bool>(0) ||\n"
-      "                       requires(T t) { t.bar(); } && sizeof(T) <= 8;");
 
----------------
What would happen with parentheses added?
```
concept DelayedCheck = (static_cast<bool>(0) || requires(T t) { t.bar(); }) && 
sizeof(T) <= 8;
concept DelayedCheck = static_cast<bool>(0) || (requires(T t) { t.bar(); } && 
sizeof(T) <= 8);
```


================
Comment at: clang/unittests/Format/FormatTest.cpp:23946-23947
-  verifyFormat("template <typename T>\n"
-               "concept C =\n"
-               "class X;");
 
----------------
Just want to make sure that the line break didn't matter.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140339

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

Reply via email to