[llvm-commits] CVS: reopt/lib/TraceToFunction/TraceToFunction.cpp

Brian Gaeke gaeke at cs.uiuc.edu
Mon May 24 15:38:03 PDT 2004


Changes in directory reopt/lib/TraceToFunction:

TraceToFunction.cpp updated: 1.57 -> 1.58

---
Log message:

Our dominance assumption for phi nodes only holds for phi nodes on the trace.


---
Diffs of the changes:  (+2 -2)

Index: reopt/lib/TraceToFunction/TraceToFunction.cpp
diff -u reopt/lib/TraceToFunction/TraceToFunction.cpp:1.57 reopt/lib/TraceToFunction/TraceToFunction.cpp:1.58
--- reopt/lib/TraceToFunction/TraceToFunction.cpp:1.57	Sat May 22 01:56:39 2004
+++ reopt/lib/TraceToFunction/TraceToFunction.cpp	Mon May 24 15:36:10 2004
@@ -113,8 +113,8 @@
     Instruction *UInst = dyn_cast<Instruction> (*ui);
     if (!UInst)                      // Non-Instruction Users scare me, mommy
       return false;
-    if (isa<PHINode> (UInst))
-      continue; // If a Phi node uses a value that's defined in the
+    if (isa<PHINode> (UInst) && T.contains (UInst->getParent ()))
+      continue; // If a Phi node in the trace uses a value that's defined in the
                 // trace, the def. must have originally dominated the Phi.
     BasicBlock *UBlock = UInst->getParent ();
     if (Block == UBlock) {





More information about the llvm-commits mailing list