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

Igor Veresov iveresov at openjdk.java.net
Wed Feb 9 23:52:09 UTC 2022


On Wed, 9 Feb 2022 23:11:42 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Revert ARM32 checks
>
> What's the reason for keeping load_immediate requiring a jlong?  It appears that all the immediates currently used will fit in a 32-bit signed int.  I suggest reverting back to "int" and simplifying this patch.

@dean-long I think the argument has to be a jlong since he's passing a long inverted mask. It has to be 64bit.

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

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


More information about the hotspot-compiler-dev mailing list