RFR: 8241502: C2: Migrate x86_64.ad to MacroAssembler [v8]
Vladimir Kozlov
kvn at openjdk.java.net
Thu Apr 22 22:20:34 UTC 2021
On Thu, 22 Apr 2021 22:05:44 GMT, John Tortugo <github.com+2249648+JohnTortugo at openjdk.org> wrote:
>> Relates to: https://bugs.openjdk.java.net/browse/JDK-8241502
>> Tested on: Linux tier1, 2 and 3
>>
>> Can you please take a look whether these changes are going in the direction expected or not? If it is, I'll continue working on the `JDK-8241502` but I'd like to split it in a few PRs since it's a lot of changes.
>
> John Tortugo has updated the pull request incrementally with one additional commit since the last revision:
>
> Fix cast of constant
src/hotspot/cpu/x86/assembler_x86.cpp line 9413:
> 9411:
> 9412: void Assembler::set_byte_if_not_zero(Register dst) {
> 9413: emit_int24(0x0F, (unsigned char)0x95, (0xC0 | dst->encoding()));
Can you explain this change?
I see it is used only in c1_LIRAssembler_x86.cpp
-------------
PR: https://git.openjdk.java.net/jdk/pull/2420
More information about the hotspot-compiler-dev
mailing list