RFR: 8290706: Remove the support for inline contiguous allocations [v2]
Aleksey Shipilev
shade at openjdk.org
Mon Jul 25 09:08:57 UTC 2022
On Mon, 25 Jul 2022 08:50:17 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:
> Looks good in general. However, the StorePConditionalNode is dead after this change, and I think we should remove that one as well while we are at it. AFAIK it is only used for the atomic bump pointer code. There is a whole bunch of StorePConditional matching rules in all the ad files that can be nuked, and a few mentions of Op_StorePConditional in C2 that can be removed.
True. I'd rather do that after this change lands, as to not invalidate the platform testing already done here. It would be also beneficial for potential bisects to have two atomic commits -- one removing the code from allocation paths, and another removing the C2 nodes and match rules. WDYT?
-------------
PR: https://git.openjdk.org/jdk/pull/9576
More information about the hotspot-dev
mailing list