I have a small problem with this - seems like we're rushing patches in without much review or discussion. For a while there we were able to test on a few platforms first, lazily, because there was more than 1 hour between patch submission and commit.

Was there any on-list discussion about this?  did I simply miss it?

geir


Salikh Zakirov wrote:
Hi,

DRLVM is broken on Linux/x86_64 (SUSE9) after following commit

 [r478172] Applied HARMONY-2223 [drlvm][jit][performance] Inlined int64
arithmetic
    routines - speedup of 64 arithmetics. Tested on SUSE9, observed ~5%
    gain on the microbenchmark.

with the following diagnostics:

/files/sszakiro/harmony/drlvm/trunk/vm/port/src/encoder/ia32_em64t/enc_tabl.cpp:1540:
static void EncoderBase::buildMnemonicDesc(const MnemonicInfo*):
Assertion `opcodesHashMap[minfo->mn][hash] == NOHASH' failed.

This assertion failure is observed on C Unit tests and when trying to
run java with any arguments.

Reply via email to