[PATCH] D129104: [Modules] Add ODR Check for concepts
Chuanqi Xu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 5 19:13:56 PDT 2022
ChuanqiXu added inline comments.
================
Comment at: clang/lib/AST/ASTContext.cpp:6532
+ const Expr *XCE = ConceptX->getConstraintExpr();
+ const Expr *YCE = ConceptY->getConstraintExpr();
+ assert(XCE && YCE && "ConceptDecl wihtout constraint expression?");
----------------
erichkeane wrote:
> This isn't necessarily valid here, you did a dyn_cast above.
Oh, my bad. I should use `cast`. We could find the style at line 6525-6526 and line 6519-6520.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129104/new/
https://reviews.llvm.org/D129104
More information about the cfe-commits
mailing list