Withdrawn: 8327522: Shenandoah: Remove unused references to satb_mark_queue_active_offset
duke
duke at openjdk.org
Thu May 9 03:29:58 UTC 2024
On Thu, 7 Mar 2024 08:00:46 GMT, Yude Lin <duke at openjdk.org> wrote:
> Removed an unused variable (trivial)
>
> Also, there is another place that uses satb_mark_queue_active_offset which is ShenandoahBarrierSetC2::verify_gc_barriers
> The current barrier pattern is different from what this code is expecting:
> If->Bool->CmpI->AndI->LoadUB->AddP->ConL(gc_state_offset)
> rather than
> If->Bool->CmpI->LoadB->AddP->ConL(marking_offset)
> However, this code isn't doing as much checking as its counterpart in G1 anyway (so I'm thinking removing the incorrect matching code altogether?) Looking forward to your suggestions.
This pull request has been closed without being integrated.
-------------
PR: https://git.openjdk.org/jdk/pull/18148
More information about the shenandoah-dev
mailing list