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

[Vladimir Kozlov]

Sounds good.

Thanks,
Vladimir

On 1/11/16 2:08 AM, Tobias Hartmann wrote:
> 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
>>