Request for reviews (S): 6791572: assert("duplicating node that's already been matched")

[Vladimir Kozlov]

I updated the fix after discussion with Tom.

http://cr.openjdk.java.net/~kvn/6791572/webrev.01

Thanks,
Vladimir

Vladimir Kozlov wrote:
> 
> http://cr.openjdk.java.net/~kvn/6791572/webrev.00
> 
> Fixed 6791572: assert("duplicating node that's already been matched")
> 
> Problem:
> AddP, LShiftX nodes and their inputs are marked as visited
> on x86 during the first address expression processing.
> AddP, LShiftX inputs will be not marked as shared since
> AddP, LShiftX will be marked as shared first.
> And this causes the problem since after AddP, LShiftX
> are folded into address expressions their inputs
> will be shared by several address expressions.
> 
> Solution:
> Delay marking nodes as visited if they are inputs
> to an address expression.
> 
> Reviewed by:
> 
> Fix verified (y/n): y, test case
> 
> Other testing:
> JPRT
>