RFR: 8287794: Reverse*VNode::Identity problem [v2]

Jatin Bhateja jbhateja at openjdk.org
Wed Jul 27 15:16:30 UTC 2022


On Mon, 25 Jul 2022 14:22:29 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> I don't think it would trigger a warning. The original warning, as I understand it, was to say that the *unpredicated* `else` branch is the same. So we were guaranteed to take either of branches, and thus the same code, irrelevant of the predicate. It is not the same here: we now have a third path, going out without entering either branch :)
>
> Ah, right. That makes sense. I still think the branches could me merged but I don't have a strong opinion.

Due to expression short circuiting and GCSE an optimizing compiler should produce same code with merged branches.

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

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


More information about the hotspot-compiler-dev mailing list