RFR: 8280003: C1: Reconsider uses of logical_and immediates in LIRGenerator::do_getObjectSize [v7]

Aleksey Shipilev shade at openjdk.java.net
Wed May 4 15:40:37 UTC 2022


On Thu, 28 Apr 2022 10:43:59 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> See the discussion in the bug.
>> 
>> Additional testing:
>>  - [x] Linux x86_64 fastdebug `java/lang/instrument`
>>  - [x] Linux x86_32 fastdebug `java/lang/instrument`
>>  - [x] Linux AArch64 fastdebug `java/lang/instrument`
>>  - [x] Linux ARM32 fastdebug `java/lang/instrument`
>>  - [x] Linux PPC64 fastdebug `java/lang/instrument`
>>  - [x] Linux x86_64 fastdebug `tier1`
>>  - [x] Linux x86_32 fastdebug `tier1`
>>  - [x] Linux AArch64 fastdebug `tier1`
>>  - [x] Linux PPC64 fastdebug `tier1`
>
> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 11 additional commits since the last revision:
> 
>  - Fix RISC-V too
>  - Merge branch 'master' into JDK-8280003-c1-logical-and
>  - Merge branch 'master' into JDK-8280003-c1-logical-and
>  - Revert ARM32 checks
>  - Merge branch 'master' into JDK-8280003-c1-logical-and
>  - Fixing failures in ARM32
>  - Merge branch 'master' into JDK-8280003-c1-logical-and
>  - Checking ARM32 code
>  - Use checked_cast<jint>
>  - Merge branch 'master' into JDK-8280003-c1-logical-and
>  - ... and 1 more: https://git.openjdk.java.net/jdk/compare/bf48758f...66448a5e

All right, this reproduces on ARM32:


$ make run-test TEST=java/lang/instrument/GetObjectSizeIntrinsicsTest.java TEST_VM_OPTS="-XX:+TieredCompilation"


This PR still resolves the issue. Therefore, I am integrating.

-------------

PR: https://git.openjdk.java.net/jdk/pull/7080


More information about the hotspot-compiler-dev mailing list