RFR: 8291466: C2: assert(false) failed: infinite loop in PhaseIterGVN::transform_old with -XX:+StressIGVN [v2]

Christian Hagedorn chagedorn at openjdk.org
Fri Aug 26 09:02:21 UTC 2022


On Thu, 25 Aug 2022 09:39:42 GMT, Pengfei Li <pli at openjdk.org> wrote:

>> Ok. Can you reword the comment to state that explicitly?
>
> Done in latest commit

Since there is now only a single assignment, I suggest to move the assignment out of the condition to make it more readable and more clear that we cannot have `con == 0` afterwards.

const jint con = in(2)->find_int_con(0);
if (con == 0) {
...

Same further down for `MulL`.

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

PR: https://git.openjdk.org/jdk/pull/9820


More information about the hotspot-compiler-dev mailing list