harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From var...@apache.org
Subject svn commit: r528397 - /harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32PeepHole.cpp
Date Fri, 13 Apr 2007 09:08:38 GMT
Author: varlax
Date: Fri Apr 13 02:08:37 2007
New Revision: 528397

URL: http://svn.apache.org/viewvc?view=rev&rev=528397
Log:
Workaround for HARMONY-3183 drlvm][jit] HelloWorld app fails in -Xem:opt mode with verification
level=2
Commented out XORPS to PXOR replacement until better solution.

Modified:
    harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32PeepHole.cpp

Modified: harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32PeepHole.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32PeepHole.cpp?view=diff&rev=528397&r1=528396&r2=528397
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32PeepHole.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32PeepHole.cpp Fri Apr
13 02:08:37 2007
@@ -633,11 +633,13 @@
           why: XORPS/PD used for zero-ing register, but PXOR is faster 
                (2 ticks on PXOR vs 4 ticks for XORPS/XORPD)
         */
-        dst = convertToXmmReg64(dst);
-        src = convertToXmmReg64(src);
-        Inst* ii = irManager->newInst(Mnemonic_PXOR, dst, src);
-        replaceInst(inst, ii);
-        return Changed_Inst;
+        // FIXME: replacing operands on 1 instruction only 
+        // will fail liveness verification if their refcount > 1
+        //dst = convertToXmmReg64(dst);
+        //src = convertToXmmReg64(src);
+        //Inst* ii = irManager->newInst(Mnemonic_PXOR, dst, src);
+        //replaceInst(inst, ii);
+        //return Changed_Inst;
     }
     return Changed_Nothing;
 }



Mime
View raw message