RFR: 8294194: [AArch64] Create intrinsics compress and expand [v2]

Andrew Dinn adinn at openjdk.org
Tue Nov 1 10:57:28 UTC 2022


On Tue, 1 Nov 2022 08:48:10 GMT, Andrew Haley <aph at openjdk.org> wrote:

>> Stuart Monteith has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update src/hotspot/cpu/aarch64/aarch64.ad
>>   
>>   Correct slight formatting error.
>>   
>>   Co-authored-by: Eric Liu <eric.c.liu at arm.com>
>
> src/hotspot/cpu/aarch64/aarch64.ad line 2294:
> 
>> 2292:     case Op_CacheWBPostSync:
>> 2293:       if (!VM_Version::supports_data_cache_line_flush()) {
>> 2294:         ret_value = false;
> 
> Doesn't this stuff belong in aarch64_sve.ad?

I'm not sure exactly what you are asking about here. However, if it is about the Op_CacheWB* checks then the answer is no. These are memory flush operations that are used by the code that syncs writes to persistent memory (NVRam) mapped buffers.

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

PR: https://git.openjdk.org/jdk/pull/10537


More information about the hotspot-compiler-dev mailing list