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