RFR: 8310264: In PhaseChaitin::Split defs and phis are leaked [v5]

Johan Sjölen jsjolen at openjdk.org
Wed Jun 28 13:34:05 UTC 2023


On Mon, 19 Jun 2023 06:20:38 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Johan Sjölen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Revert to list init
>
> Looks good! I'm wondering, why we don't stack allocate both `Node_List` instead of `new Node_List()`. But regardless of that, we should indeed add a `ResourceMark`.

@chhagedorn , @TobiHartmann , would you mind re-approving this PR? Thank you.

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

PR Comment: https://git.openjdk.org/jdk/pull/14530#issuecomment-1611419407


More information about the hotspot-compiler-dev mailing list