You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@harmony.apache.org by va...@apache.org on 2006/11/16 14:42:02 UTC

svn commit: r475732 - /incubator/harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/ssa/SSA.cpp

Author: varlax
Date: Thu Nov 16 05:42:01 2006
New Revision: 475732

URL: http://svn.apache.org/viewvc?view=rev&rev=475732
Log:
Follow-up comment for HARMONY-1905 added

Modified:
    incubator/harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/ssa/SSA.cpp

Modified: incubator/harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/ssa/SSA.cpp
URL: http://svn.apache.org/viewvc/incubator/harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/ssa/SSA.cpp?view=diff&rev=475732&r1=475731&r2=475732
==============================================================================
--- incubator/harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/ssa/SSA.cpp (original)
+++ incubator/harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/ssa/SSA.cpp Thu Nov 16 05:42:01 2006
@@ -106,6 +106,10 @@
 
 // Checks that phi insts can start from the second or third position only
 // and goes in a row
+// The first instruction is usually a LabelInst.
+// The second one is Phi (if any). But in catch blocks the second inst
+// is a CatchInst, and phi may be shifted to the third position.
+
 bool SSABuilder::phiInstsOnRightPositionsInBB(Node* node) {
     Inst* inst = (Inst*)node->getSecondInst();
     if(inst && !inst->isPhi()) {