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 2007/04/02 14:25:45 UTC

svn commit: r524773 - in /harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer: FastArrayFilling.cpp escanalyzer.cpp

Author: varlax
Date: Mon Apr  2 05:25:42 2007
New Revision: 524773

URL: http://svn.apache.org/viewvc?view=rev&rev=524773
Log:
Few unused variables dropped to make Intel compiler (ICL9) happy.

Modified:
    harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/FastArrayFilling.cpp
    harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/escanalyzer.cpp

Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/FastArrayFilling.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/FastArrayFilling.cpp?view=diff&rev=524773&r1=524772&r2=524773
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/FastArrayFilling.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/FastArrayFilling.cpp Mon Apr  2 05:25:42 2007
@@ -100,7 +100,6 @@
         Opnd* inc = NULL;
         Opnd * arrayBound = NULL;
         Opnd * fillBound = NULL;
-        Opnd *startIndex;
         Inst * inst = ((Inst *)startNode->getLastInst());
         bool found = false;
 
@@ -174,7 +173,6 @@
 
         //check StVar
         if (inst->getOpcode() == Op_StVar && inst->getDst() == index) {
-            startIndex = inst->getSrc(0);
             inst = inst->getPrevInst();
             //check StInd
             if (inst->getOpcode() == Op_TauStInd && inst->getSrc(0) == constValue && inst->getSrc(1) == arrayBase) {
@@ -232,7 +230,6 @@
                 break;
             default:
                 continue;
-                break;
         }
 
         ControlFlowGraph& fg = irManager.getFlowGraph();

Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/escanalyzer.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/escanalyzer.cpp?view=diff&rev=524773&r1=524772&r2=524773
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/escanalyzer.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/optimizer/escanalyzer.cpp Mon Apr  2 05:25:42 2007
@@ -4791,8 +4791,6 @@
 EscAnalyzer::doLOScalarReplacement(ObjIds* loids) {
     ObjIds::iterator lo_it;
     CnGNode* onode;
-    Opnd* oopnd;
-    Type* otype;
     Inst* inst;
     Inst* st_inst;
     Insts::iterator it3;
@@ -4817,8 +4815,6 @@
         if (onode->nodeRefType == NR_PRIM) {
             continue;
         }
-        oopnd = (Opnd*)(onode->refObj);
-        otype = oopnd->getType();
         if (_scinfo) {
             os_sc << " - - method: ";
             irManager.getMethodDesc().printFullName(os_sc); 
@@ -4871,7 +4867,6 @@
                     VarOpnd* fl_var_opnd = _opndManager.createVarOpnd(fl_type, false);
                     SsaTmpOpnd* fl_init_opnd = _opndManager.createSsaTmpOpnd(fl_type);
                     Inst* ld_init_val_inst = NULL;
-                    Inst* st_init_val_inst = NULL;
                     sco->fldVarOpnd = fl_var_opnd;
 
                     if (_scinfo) {
@@ -4882,7 +4877,6 @@
                     } else {
                         ld_init_val_inst = _instFactory.makeLdConst(fl_init_opnd, 0);
                     }
-                    st_init_val_inst = _instFactory.makeStVar(fl_var_opnd, fl_init_opnd);
 
                     scalarizeOFldUsage(sco);
                     if (_scinfo) {
@@ -4945,8 +4939,6 @@
 EscAnalyzer::doEOScalarReplacement(ObjIds* loids) {
     ObjIds::iterator lo_it;
     CnGNode* onode;
-    Opnd* oopnd;
-    Type* otype;
     Inst* inst;
     Insts::iterator it3;
     OpndManager& _opndManager = irManager.getOpndManager();
@@ -4980,8 +4972,6 @@
         } else {
             lobj_opt = true;
         }
-        oopnd = (Opnd*)(onode->refObj);
-        otype = oopnd->getType();
         double path_prob = -1;
         path_prob = checkLocalPath(onode->nInst);
         os_sc<<"pp      " << (path_prob )<<std::endl;
@@ -5182,7 +5172,6 @@
                 VarOpnd* fl_var_opnd = _opndManager.createVarOpnd(fl_type, false);
                 SsaTmpOpnd* fl_init_opnd = _opndManager.createSsaTmpOpnd(fl_type);
                 Inst* ld_init_val_inst = NULL;
-                Inst* st_init_val_inst = NULL;
                 sco->fldVarOpnd = fl_var_opnd;
 
                 if (_scinfo) {
@@ -5193,7 +5182,6 @@
                 } else {
                     ld_init_val_inst = _instFactory.makeLdConst(fl_init_opnd, 0);
                 }
-                st_init_val_inst = _instFactory.makeStVar(fl_var_opnd, fl_init_opnd);
                 scalarizeOFldUsage(sco);
                 if (_scinfo) {
                     os_sc << "++++ old newobj added fld_var: before"  << std::endl;
@@ -5462,7 +5450,6 @@
 EscAnalyzer::checkOpndUsage(ObjIds* lnoids, ObjIds* lloids, bool check_loc) {
     const Nodes& nodes = irManager.getFlowGraph().getNodes();
     Nodes::const_iterator niter;
-    bool maydo = true;
     bool do_break = false;
 
     if (_scinfo) {
@@ -5498,7 +5485,6 @@
                         if (opcode == Op_IndirectMemoryCall || opcode == Op_DirectCall)
                             continue;
 
-                    maydo = false;
                     if (_scinfo) {
                         os_sc << " no   remove " << opnd->getId() << std::endl; 
                     }
@@ -5525,7 +5511,6 @@
                             continue;
                         }
                     }
-                    maydo = false;
                     if (_scinfo) {
                         os_sc << " lo   remove " << opnd->getId() << std::endl; 
                     }