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

Vladimir Kozlov vladimir.kozlov at oracle.com
Sat Jan 18 13:02:52 PST 2014


immU32 name is not correct, it should be immU31.

Vladimir K

On 1/18/14 12:51 PM, Igor Veresov wrote:
> 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
>


More information about the hotspot-compiler-dev mailing list