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

Tobias Hartmann tobias.hartmann at oracle.com
Fri Jan 8 08:03:47 UTC 2016


Thanks for the review, Vladimir.

Best,
Tobias

On 07.01.2016 20:08, Vladimir Kozlov wrote:
> Perfect.
> 
> Thanks,
> Vladimir
> 
> On 1/7/16 10:51 AM, 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