[9] RFR(M): 8146629: Make phase->is_IterGVN() accessible from Node::Identity and Node::Value

Tobias Hartmann tobias.hartmann at oracle.com
Mon Jan 11 10:08:31 UTC 2016


FYI, I had to merge with JDK-8143353 [1] (CosDNode and SinDNode were removed).

This is the change I indent to push:
http://cr.openjdk.java.net/~thartmann/8146629/webrev.01/

Thanks,
Tobias

[1] http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/13b04370e8e9

On 07.01.2016 19:51, Tobias Hartmann wrote:
> Hi,
> 
> please review the following patch.
> 
> https://bugs.openjdk.java.net/browse/JDK-8146629
> http://cr.openjdk.java.net/~thartmann/8146629/webrev.00/
> 
> Currently, there is no way to determine in Node::Identity() and Node::Value() if we were called from GVN or IGVN but sometimes we would like to do optimizations based on this information (for example, see discussion in RFR for JDK-8136469 [1]). I changed the arguments of Node::Identity() and Node::Value() from PhaseTransform* to PhaseGVN*. Like this, we can simply call PhaseValues::is_IterGVN() from both methods.
> 
> Thanks,
> Tobias
> 
> [1] http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-January/020670.html
> 


More information about the hotspot-compiler-dev mailing list