commit ef784f01dd5981b95c634c3d2cf1c64dfce0fcc4
author Reid Spencer <rspencer@reidspencer.com> Sun Jul 18 00:32:14 2004 +0000
committer Reid Spencer <rspencer@reidspencer.com> Sun Jul 18 00:32:14 2004 +0000
tree 685f8b06a5e7987b95929e50fddab2147ab190e3
parent c5afc9512bfe9d5c1695d064be940d7cf58e8e1d

bug 122:
- Minimize redundant isa<GlobalValue> usage

llvm-svn: 14948

llvm/lib/Transforms/Scalar/GCSE.cpp

diff --git a/llvm/lib/Transforms/Scalar/GCSE.cpp b/llvm/lib/Transforms/Scalar/GCSE.cpp
index 7d93c50..c249413 100644
--- a/llvm/lib/Transforms/Scalar/GCSE.cpp
+++ b/llvm/lib/Transforms/Scalar/GCSE.cpp
@@ -78,8 +78,7 @@
       VN.getEqualNumberNodes(AI, EqualValues);
       if (!EqualValues.empty()) {
         for (unsigned i = 0, e = EqualValues.size(); i != e; ++i)
-          if (isa<Constant>(EqualValues[i]) ||
-              isa<GlobalValue>(EqualValues[i])) {
+          if (isa<Constant>(EqualValues[i])) {
             AI->replaceAllUsesWith(EqualValues[i]);
             ++NumArgsRepl;
             Changed = true;
@@ -186,7 +185,7 @@
 
   // If we are not replacing the instruction with a constant, we cannot do
   // anything special.
-  if (!isa<Constant>(V)) {
+  if (!isa<Constant>(V) || isa<GlobalValue>(V)) {
     I->replaceAllUsesWith(V);
 
     if (InvokeInst *II = dyn_cast<InvokeInst>(I)) {