RFR: 8290706: Remove the support for inline contiguous allocations [v2]
Andrew Haley
aph at openjdk.org
Mon Jul 25 14:03:10 UTC 2022
On Mon, 25 Jul 2022 09:05:16 GMT, Aleksey Shipilev <shade 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?
Yes, I agree. There's more cruft in there.
-------------
PR: https://git.openjdk.org/jdk/pull/9576
More information about the hotspot-dev
mailing list