RFR: 8373591: C2: Fix the memory around some intrinsics nodes [v3]

Quan Anh Mai qamai at openjdk.org
Tue Dec 16 17:36:21 UTC 2025


On Tue, 16 Dec 2025 16:51:27 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>> Quan Anh Mai has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use MemBar instead of widening the intrinsic memory
>
> src/hotspot/share/opto/graphKit.cpp line 4210:
> 
>> 4208:     //  StoreC -> MemBar -> MergeMem -> compress_string -> MergeMem -> CharMem
>> 4209:     //                               -------------------------------->
>> 4210:     Node* all_mem = reset_memory();
> 
> This code sequence is used several times. Would it make sense to factor it out in its own method?

Done!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28789#discussion_r2624148395


More information about the hotspot-compiler-dev mailing list