diff --git a/clang/lib/AST/ExprConstant.cpp b/clang/lib/AST/ExprConstant.cpp --- a/clang/lib/AST/ExprConstant.cpp +++ b/clang/lib/AST/ExprConstant.cpp @@ -8631,7 +8631,7 @@ 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 --- a/clang/test/SemaCXX/recovery-expr-type.cpp +++ b/clang/test/SemaCXX/recovery-expr-type.cpp @@ -151,3 +151,13 @@ // 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 +} +}