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

[Tobias Hartmann]

I had to merge again with JDK-8139771 (castnode.cpp/hpp):
http://cr.openjdk.java.net/~thartmann/8146629/webrev.02/

Thanks,
Tobias

On 11.01.2016 20:30, Vladimir Kozlov wrote:
> 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
>>>