[master] RFR: Load Klass from header, C1 [v10]
Roman Kennke
rkennke at openjdk.java.net
Tue Jan 11 17:22:29 UTC 2022
On Tue, 11 Jan 2022 16:46:10 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Roman Kennke has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 38 commits:
>>
>> - Merge branch 'master' into klass-from-header-c1
>> - AArch64 implementation
>> - Merge remote-tracking branch 'origin/klass-from-header-c1' into klass-from-header-c1
>> - Merge remote-tracking branch 'upstream/master' into klass-from-header-c1
>> - Increase code size estimate
>> - Merge branch 'master' into klass-from-header-c1
>> - Merge remote-tracking branch 'jdk-upstream/master' into klass-from-header-c1
>> - Merge remote-tracking branch 'upstream/master' into klass-from-header-c1
>> - Basic arm implementation
>> - Basic s390 implementation
>> - ... and 28 more: https://git.openjdk.java.net/lilliput/compare/e8ff7a0f...6136cf94
>
> src/hotspot/cpu/x86/c1_LIRAssembler_x86.cpp line 3531:
>
>> 3529: Register tmp = rscratch1;
>> 3530: assert_different_registers(tmp, obj);
>> 3531: assert_different_registers(tmp, result);
>
> Wait, so this thing would not be executed for `!_LP64` anymore?
Indeed! Good catch! Will fix before push.
-------------
PR: https://git.openjdk.java.net/lilliput/pull/20
More information about the lilliput-dev
mailing list