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