RFR: 8317349: Randomize order of macro node expansion in C2 [v4]

Vladimir Kozlov kvn at openjdk.org
Tue Feb 6 18:21:54 UTC 2024


On Mon, 5 Feb 2024 15:08:43 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

> Looks good to me, modulo Christian's comments. Regarding whether to randomize macro node elimination, I find it a bit confusing that `PhaseMacroExpand` also eliminates macro nodes, perhaps it would be better to, in a separate RFE, split the phase into `PhaseMacroEliminate` and `PhaseMacroExpand`, and introduce a separate `StressMacroElimination` option and new phase names for the former?

Yes, I agree with this.

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

PR Comment: https://git.openjdk.org/jdk/pull/17684#issuecomment-1930512131


More information about the hotspot-compiler-dev mailing list