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

Thanks!



================
Comment at: clang/include/clang/AST/RecursiveASTVisitor.h:1781
+  if (const auto *TC = D->getTypeConstraint()) {
+    Expr *IDC = TC->getImmediatelyDeclaredConstraint();
+    TRY_TO(TraverseStmt(IDC));
----------------
nit: just 

```
if (Expr *IDC = ...) {
  ...
} else {
    TRY_TO(TraverseConceptReference(*TC));
}
```


================
Comment at: clang/unittests/Tooling/RecursiveASTVisitorTests/Concept.cpp:2
+//===- unittest/Tooling/RecursiveASTVisitorTests/Concept.cpp
+//----------------===//
+//
----------------
nit: should be merged in the above line.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84136

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

Reply via email to