RFR(S) 8031743: C2: loadI2L_immI broken for negative memory values (original) (raw)

Igor Veresov igor.veresov at oracle.com
Sat Jan 18 12:51:35 PST 2014

• Previous message: RFR(XXS): 8032207 C2: assert(VerifyOops || MachNode::size(ra_) <= (3+1)*4) failed: bad fixed size
• Next message: RFR(S) 8031743: C2: loadI2L_immI broken for negative memory values
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Transformation of (ConvI2L (AndI (LoadI mem) mask)) to (AndI (LoadUI2L (mem) mask) is only valid for positive mask, otherwise the sign extending effect of ConvI2L is missed. The solution is to restrict the optimization for the positive values of mask.

Webrev: http://cr.openjdk.java.net/~iveresov/8031743/webrev.00/ Testing: new regression test, jprt

Thanks! igor

• Previous message: RFR(XXS): 8032207 C2: assert(VerifyOops || MachNode::size(ra_) <= (3+1)*4) failed: bad fixed size
• Next message: RFR(S) 8031743: C2: loadI2L_immI broken for negative memory values
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the hotspot-compiler-dev mailing list