diff options
| author | Chris Lattner <sabre@nondot.org> | 2008-12-15 03:46:38 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2008-12-15 03:46:38 +0000 |
| commit | f33131685bb9de34dd876641bdf2d514fb49ce4c (patch) | |
| tree | b310c95ae993b5a41e46019a975931ec910062c1 /lib | |
| parent | 9e59c64c14cfe55e7cc9086c6bff8cfeecac361e (diff) | |
| download | external_llvm-f33131685bb9de34dd876641bdf2d514fb49ce4c.zip external_llvm-f33131685bb9de34dd876641bdf2d514fb49ce4c.tar.gz external_llvm-f33131685bb9de34dd876641bdf2d514fb49ce4c.tar.bz2 | |
make GVN try to rename inputs to the resultant replaced values, which
cleans up the generated code a bit. This should have the added benefit of
not randomly renaming functions/globals like my previous patch did. :)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61023 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/Transforms/Scalar/GVN.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/GVN.cpp b/lib/Transforms/Scalar/GVN.cpp index 8cfc627..d3e15d4 100644 --- a/lib/Transforms/Scalar/GVN.cpp +++ b/lib/Transforms/Scalar/GVN.cpp @@ -1040,6 +1040,9 @@ bool GVN::processNonLocalLoad(LoadInst *LI, // Perform PHI construction. Value* v = GetValueForBlock(LI->getParent(), LI, BlockReplValues, true); LI->replaceAllUsesWith(v); + + if (!isa<GlobalValue>(v)) + v->takeName(LI); if (isa<PointerType>(v->getType())) MD->invalidateCachedPointerInfo(v); toErase.push_back(LI); @@ -1135,7 +1138,8 @@ bool GVN::processNonLocalLoad(LoadInst *LI, // Perform PHI construction. Value* v = GetValueForBlock(LI->getParent(), LI, BlockReplValues, true); LI->replaceAllUsesWith(v); - v->takeName(LI); + if (!isa<GlobalValue>(v)) + v->takeName(LI); if (isa<PointerType>(v->getType())) MD->invalidateCachedPointerInfo(v); toErase.push_back(LI); |