[14] RFR(S): 8233656: assert(d->is_CFG() && n->is_CFG()) failed: must have CFG nodes

[Vladimir Kozlov]

+1 (good)

Thanks,
Vladimir K.

On 11/12/19 3:00 AM, Vladimir Ivanov wrote:
> 
>> http://cr.openjdk.java.net/~thartmann/8233656/webrev.02/
> 
> Looks good. (And sorry for the misleading suggestion.)
> 
> Best regards,
> Vladimir Ivanov
> 
>> On 12.11.19 11:49, Tobias Hartmann wrote:
>>> Okay, this is actually not correct:
>>> If d is an IfTrue projection, we would change d to the corresponding If node which could be a
>>> dominator of n while the IfTrue projection is not.
>>>
>>> Best regards,
>>> Tobias
>>>
>>> On 12.11.19 10:52, Tobias Hartmann wrote:
>>>> Thanks Vladimir!
>>>>
>>>> Best regards,
>>>> Tobias
>>>>
>>>> On 12.11.19 10:52, Vladimir Ivanov wrote:
>>>>>
>>>>>> http://cr.openjdk.java.net/~thartmann/8233656/webrev.01/
>>>>>
>>>>> Looks good.
>>>>>
>>>>> Best regards,
>>>>> Vladimir Ivanov