diff options
author | Chandler Carruth <chandlerc@gmail.com> | 2012-08-07 10:59:59 +0000 |
---|---|---|
committer | Chandler Carruth <chandlerc@gmail.com> | 2012-08-07 10:59:59 +0000 |
commit | 961e1acfb275613679c0d00d4a0b4ed394b51a9d (patch) | |
tree | 09ed55ad41b901e58f64a5837626733e7a10b725 /Makefile | |
parent | e6450dc2afc18531bf9b70180a9f67376d9f00c7 (diff) | |
download | external_llvm-961e1acfb275613679c0d00d4a0b4ed394b51a9d.zip external_llvm-961e1acfb275613679c0d00d4a0b4ed394b51a9d.tar.gz external_llvm-961e1acfb275613679c0d00d4a0b4ed394b51a9d.tar.bz2 |
Fix PR13412, a nasty miscompile due to the interleaved
instsimplify+inline strategy.
The crux of the problem is that instsimplify was reasonably relying on
an invariant that is true within any single function, but is no longer
true mid-inline the way we use it. This invariant is that an argument
pointer != a local (alloca) pointer.
The fix is really light weight though, and allows instsimplify to be
resiliant to these situations: when checking the relation ships to
function arguments, ensure that the argumets come from the same
function. If they come from different functions, then none of these
assumptions hold. All credit to Benjamin Kramer for coming up with this
clever solution to the problem.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@161410 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'Makefile')
0 files changed, 0 insertions, 0 deletions