RFR: 8329261: G1: interpreter post-barrier x86 code asserts index size of wrong buffer

Axel Boldt-Christmas aboldtch at openjdk.org
Thu Apr 4 09:25:00 UTC 2024


On Thu, 4 Apr 2024 08:36:37 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

> This changeset updates an assert in G1's interpreter x86 post-barrier logic so that it refers to the right queue (`G1DirtyCardQueue` rather than pre-barrier's `SATBMarkQueue`) and moves the assert closer to the logic that exploits it.
> 
> Thanks to Kim Barrett for reporting the issue and suggesting the fix.
> 
> **Testing**: built on windows-x64, linux-x64, and macosx-x64.

Looks good. 

It is a little strange to me to use `pointer sized`, `sizeof(intptr_t)` and `wordSize` this interchangeably. 

Side note: The assert that used the correct queue type (in `generate_c1_pre_barrier_runtime_stub`) now have different styles w.r.t. having the static assert at the top of the function vs. next to the specific logic.

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

Marked as reviewed by aboldtch (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18616#pullrequestreview-1979288356


More information about the hotspot-dev mailing list