diff --git a/clang/lib/AST/ExprConstant.cpp b/clang/lib/AST/ExprConstant.cpp index 3ad44d5e2da808..70a1e0bbd91aae 100644 --- a/clang/lib/AST/ExprConstant.cpp +++ b/clang/lib/AST/ExprConstant.cpp @@ -8631,7 +8631,7 @@ static bool evaluateLValueAsAllocSize(EvalInfo &Info, APValue::LValueBase Base, return false; const Expr *Init = VD->getAnyInitializer(); - if (!Init) + if (!Init || Init->getType().isNull()) return false; const Expr *E = Init->IgnoreParens(); diff --git a/clang/test/SemaCXX/recovery-expr-type.cpp b/clang/test/SemaCXX/recovery-expr-type.cpp index 94b275c9a362fd..3db11466529e15 100644 --- a/clang/test/SemaCXX/recovery-expr-type.cpp +++ b/clang/test/SemaCXX/recovery-expr-type.cpp @@ -151,3 +151,13 @@ enum Circular { // expected-note {{not complete until the closing '} // Enumerators can be evaluated (they evaluate as zero, but we don't care). static_assert(Circular_A == 0 && Circular_A != 0, ""); // expected-error {{static_assert failed}} } + +namespace test14 { +extern "C" void *memset(void *, int b, unsigned long) { + int * const c(undef()); // expected-error {{undeclared identifier}} + // Verify we do not crash on evaluating *c whose initializer is a NULL-type ParenListExpr! + memset(c, 0, *c); // crash1 + + b = __builtin_object_size(c, 0); // crash2 +} +}