RFR: 8252583: Make PhiNode::is_copy() debug only [v2]

Vladimir Kozlov kvn at openjdk.java.net
Mon Sep 21 20:54:42 UTC 2020


On Mon, 21 Sep 2020 12:32:36 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Roberto Castañeda Lozano has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix spacing in touched lines
>
> Changes requested by thartmann (Reviewer).

Sorry, I am not sure this is better than original code where assert is in one place - in PhiNode::is_copy().
The method is in .hpp file and is inlined - NULL check will be eliminated. It will only be executed in slowdebug build.
May be we should "bite the bullet" and remove this method at all - we don't hit the assert in years.
We can replace is_copy() check with assert in PhiNode::Ideal() - that should be enough to guarantee that we don't
create Phi with NULL control edge.

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

PR: https://git.openjdk.java.net/jdk/pull/275


More information about the hotspot-compiler-dev mailing list