[PATCH] D76504: [clang] Fix crash during template sema checking

Clement Courbet via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 20 09:11:04 PDT 2020


courbet added inline comments.


================
Comment at: clang/lib/Sema/SemaChecking.cpp:1655
+    clang::Expr *SizeOp = TheCall->getArg(2);
+    // If any arg is instantiation dependent we bail out.
+    if (DstOp->isInstantiationDependent() ||
----------------
Are the first two necessary ? IIRC only `EvaluateKnownConstInt()` does not like template-dependent stuff. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76504





More information about the cfe-commits mailing list