RFR: 8346280: C2: implement late barrier elision for G1 [v2]

Thomas Schatzl tschatzl at openjdk.org
Fri Jan 31 08:57:52 UTC 2025


On Fri, 31 Jan 2025 08:48:52 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Roberto Castañeda Lozano has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 11 commits:
>> 
>>  - Merge commit 'afcc2b0' into JDK-8346280-late-barrier-elision
>>  - Add new test case (store after exception)
>>  - Remove unused includes
>>  - Remove temporary UseNewCode guard
>>  - Use clearer names and document access API interface
>>  - Move ZGC-specific barrier elision code to BarrierSetC2
>>  - Set UseNewCode to true temporarily
>>  - Make elide_mach_barrier virtual
>>  - Move late barrier elision analysis from G1BarrierSetC2 to BarrierSetC2
>>  - Initial implementation
>>  - ... and 1 more: https://git.openjdk.org/jdk/compare/afcc2b03...ab47d7f8
>
> src/hotspot/share/gc/shared/c2/barrierSetC2.cpp line 956:
> 
>> 954: static bool is_unknown(intptr_t offset) {
>> 955:   return offset == Type::OffsetBot;
>> 956: }
> 
> pre-existing: these two predicates would be nice to have elsewhere in C2 code too (i.e. there's quite a lot of these predicates inlined elsewhere).

Maybe something for another RFE of course.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23235#discussion_r1936890465


More information about the hotspot-compiler-dev mailing list