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

Johan Sjölen jsjolen at openjdk.org
Wed Jun 28 12:38:03 UTC 2023


On Mon, 26 Jun 2023 19:34:16 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Thanks for the background, I wasn't aware of that. I don't have a strong opinion but consistency in the same area would be nice.
>
> Please, change to normal `()`. Using '{}' is very confusing for not modern C++ experts and affects maintainability of this code.

Thanks for the input, reverted to `()`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14530#discussion_r1245142713


More information about the hotspot-compiler-dev mailing list