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