diff options
| author | Bill Wendling <isanbard@gmail.com> | 2012-05-08 18:22:25 +0000 |
|---|---|---|
| committer | Bill Wendling <isanbard@gmail.com> | 2012-05-08 18:22:25 +0000 |
| commit | 82520fb52345e8a35ede5cce650ae5b4f6778d21 (patch) | |
| tree | e3e0b4aae40828ee5553722ef8ad294e6ecdb036 | |
| parent | eee8b7c1f0a72dd71918928298a82e80cd13bc93 (diff) | |
Merging r155803:
------------------------------------------------------------------------
r155803 | rsmith | 2012-04-29 17:16:51 -0700 (Sun, 29 Apr 2012) | 8 lines
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/branches/release_31@156394 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Analysis/UninitializedValues.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/Analysis/UninitializedValues.cpp b/lib/Analysis/UninitializedValues.cpp index 6e5da25259..1c7e6b62f8 100644 --- a/lib/Analysis/UninitializedValues.cpp +++ b/lib/Analysis/UninitializedValues.cpp @@ -168,7 +168,8 @@ static const BinaryOperator *getLogicalOperatorInChain(const CFGBlock *block) { 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; |