Request for reviews (XS): 7043552: regression after 7042327

[Vladimir Kozlov]

Yes, one after an other for each unroll:

0e0     movl    R11, #-2147483648       # int
0e6     cmpl    R10, [RSP + #8 (32-bit)]
0eb     movl    R8, [rsp + #8]  # spill
0f0     cmovll R8, R11  # signed, int
0f4     movl    RCX, R8 # spill
0f7     addl    RCX, #-2        # int
0fa     cmpl    R8, RCX
0fd     cmovll RCX, R11 # signed, int
101     movl    R8, RCX # spill
104     addl    R8, #-4 # int
108     cmpl    RCX, R8
10b     cmovll R8, R11  # signed, int
10f     movl    RCX, R8 # spill
112     addl    RCX, #-8        # int
115     cmpl    R8, RCX
118     cmovll RCX, R11 # signed, int

Vladimir

Tom Rodriguez wrote:
> So the old code ended up stacking them up?
> 
> tom
> 
> On May 10, 2011, at 12:06 PM, Vladimir Kozlov wrote:
> 
>> http://cr.openjdk.java.net/~kvn/7043552/webrev
>>
>> Fixed 7043552: regression after 7042327
>>
>> I screw up the fix for 7042327 and Opaque2 node is generated for each unroll preventing folding CMoveI nodes. Generate Opaque2 node only during first unroll.
>>
>> Tested with failed case from 7042327.
>