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/13 11:08:38 UTC
svn commit: r528397 -
/harmony/enhanced/drlvm/trunk/vm/jitrino/src/codegenerator/ia32/Ia32PeepHole.cpp
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;
}