RFR: 8268858: Determine register pressure automatically by the number of available registers for allocation [v3]

Dean Long dlong at openjdk.java.net
Wed Jun 23 03:47:29 UTC 2021


On Tue, 22 Jun 2021 08:00:04 GMT, Joshua Zhu <jzhu at openjdk.org> wrote:

>> Hi,
>> 
>> This patch includes the following changes:
>> 
>> 1) Determine register pressure automatically by the number of available
>>    registers for allocation.
>>    Currently, INTPRESSURE is hard-coded no matter whether heapbase
>>    or framepointer register is available for RA.
>> 
>> 2) x86: Machines with AVX3 have 2x more XMM registers.
>>         The pressure threshold should be improved for both RA and LCM.
>>         Currently, only scheduling takes more XMM registers into
>>         consideration.
>> 
>> 3) aarch64: add a new jtreg case
>>         In bug 8183543, this case caused many compilation failures with
>>         "failed spill-split-recycle sanity check". This test case is
>>         imported to ensure no regression failure.
>> 
>> Inconsistent pressure threshold value with the number of available
>> registers will bring unnecessary spilling in some scenarios.
>> I wrote a simple example as one case:
>>   https://cr.openjdk.java.net/~jzhu/8268858/Test.java
>> 
>> With this patch, 4 extra spill copies are eliminated on AArch64.
>>   old OptoAssembly: https://cr.openjdk.java.net/~jzhu/8268858/old_opto_assembly
>>   new OptoAssembly: https://cr.openjdk.java.net/~jzhu/8268858/new_opto_assembly
>> 
>> Could you please help review this patch?
>> 
>> Best Regards,
>> Joshua
>
> Joshua Zhu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add more details in comments

Marked as reviewed by dlong (Reviewer).

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

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


More information about the hotspot-compiler-dev mailing list