harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Astapchuk <alex.astapc...@gmail.com>
Subject Re: [drlvm][x86_64] DRLVM broken after HARMONY-2223
Date Thu, 23 Nov 2006 06:12:53 GMT
Geir Magnusson Jr. wrote:
> 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.

Actually, the patch was there for a week - since Nov.17.

Thought, it's still my fault that I did not check the em64t build :-(
Sorry about that, I'm in working on the fix right now.

-- 
Thanks,
   Alex

> 
> 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.
>>
> 




Mime
View raw message