RFR: 8241095: x86: Improve prefix handling in Assembler
Claes Redestad
claes.redestad at oracle.com
Wed Mar 18 21:13:26 UTC 2020
On 2020-03-18 20:30, Vladimir Kozlov wrote:
> Looks good.
Thanks!
>
> I checked that moved methods stay the same. You only modified
> prefixq(Address adr) and prefixq(Address adr, Register src) as we
> discussed. Good.
Yep.
Also: By passing in rax (or any other valid register constant in the 0-7
encoding range) I can reuse get_prefixq(Address adr, Register src)
from prefixq(Address adr) without any overhead in compiled code. I'll
add a short comment on that.
/Claes
>
> Thanks,
> Vladimir
>
> On 3/18/20 6:46 AM, Claes Redestad wrote:
>> Hej,
>>
>> this patch enable folding emitted prefixq bytes into multi-byte emitter
>> functions (emit_int16(int8, int8) etc).
>>
>> Only those functions where there is always a prefix being emitted have
>> been adjusted. Other variants could improve slightly by doing similar
>> tricks, but with a smaller payoff and more complexity.
>>
>> Webrev: http://cr.openjdk.java.net/~redestad/8241095/open.00
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8241095
>>
>> Testing: tier1-4 (ongoing)
>>
>> Thanks,
>>
>> /Claes
More information about the hotspot-compiler-dev
mailing list