RFR: 8328085: C2: Use after free in PhaseChaitin::Register_Allocate()

[Richard Reingruber]

On Wed, 20 Nov 2024 10:36:38 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> Nice analysis! The fix looks reasonable to me but I'm a bit worried that such removals of ResourceMarks will lead to an increase in peak memory consumption because memory is only released much later now. And I would assume there is a reason for the ResourceMark placement, i.e., below code doing significant temporary allocations. Kind of related: [JDK-8337015](https://bugs.openjdk.org/browse/JDK-8337015).

I was also worried about increased memory consumption. There's at least the minimal testing I'm reporting about in the pr synopsis. Do you know about work loads that lead to higher memory consumption? Maybe you've got some internal tests. For that purpose I put the trace code into the pr. Should be easy to use.

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

PR Comment: https://git.openjdk.org/jdk/pull/22200#issuecomment-2488755471