commit b86b8551ce619450b2eb4fc708ead08b83996da7
author Richard Smith <richard-llvm@metafoo.co.uk> Mon Apr 30 00:16:51 2012 +0000
committer Richard Smith <richard-llvm@metafoo.co.uk> Mon Apr 30 00:16:51 2012 +0000
tree c88874bfa16e5022ddd1311311c8faf95bd78417
parent 6850fafd27ba804d4d4ca8af404beed5574e3749

PR11926 + duplicates: Fix crash in -Wuninitialized when using a compiler like
g++4.7, which reuses stack space allocated for temporaries. CFGElement::getAs
returns a suitably-cast version of 'this'. Patch by Markus Trippelsdorf!

No test: this code has the same observable behavior as the old code when built
with most compilers, and the tests were already failing when built with a
compiler for which this produced a broken binary.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@155803 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Analysis/UninitializedValues.cpp

diff --git a/lib/Analysis/UninitializedValues.cpp b/lib/Analysis/UninitializedValues.cpp
index 6e5da25..1c7e6b6 100644
--- a/lib/Analysis/UninitializedValues.cpp
+++ b/lib/Analysis/UninitializedValues.cpp
@@ -168,7 +168,8 @@
   if (block->empty())
     return 0;
 
-  const CFGStmt *cstmt = block->front().getAs<CFGStmt>();
+  CFGElement front = block->front();
+  const CFGStmt *cstmt = front.getAs<CFGStmt>();
   if (!cstmt)
     return 0;