Index: cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp =================================================================== --- cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp +++ cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp @@ -65,7 +65,7 @@ if (Optional L = V.getAs()) V = Pred->getState()->getSVal(*L); else - assert(V.isUnknown()); + assert(V.isUnknownOrUndef()); const Expr *CallExpr = Call.getOriginExpr(); evalBind(Dst, CallExpr, Pred, ThisVal, V, true); Index: cfe/trunk/test/Analysis/uninit-vals.cpp =================================================================== --- cfe/trunk/test/Analysis/uninit-vals.cpp +++ cfe/trunk/test/Analysis/uninit-vals.cpp @@ -0,0 +1,34 @@ +// RUN: %clang_cc1 -analyze -analyzer-checker=core.builtin -verify -DCHECK_FOR_CRASH %s +// RUN: %clang_cc1 -analyze -analyzer-checker=core -verify %s + +#ifdef CHECK_FOR_CRASH +// expected-no-diagnostics +#endif + +namespace PerformTrivialCopyForUndefs { +struct A { + int x; +}; + +struct B { + A a; +}; + +struct C { + B b; +}; + +void foo() { + C c1; + C *c2; +#ifdef CHECK_FOR_CRASH + // If the value of variable is not defined and checkers that check undefined + // values are not enabled, performTrivialCopy should be able to handle the + // case with undefined values, too. + c1.b.a = c2->b.a; +#else + c1.b.a = c2->b.a; // expected-warning{{Function call argument is an uninitialized value}} +#endif +} +} +