Request for reviews (L): 6823354: Add intrinsics for{Integer,Long}.{numberOfLeadingZeros,numberOfTrailingZeros}()
Vladimir Kozlov
Vladimir.Kozlov at Sun.COM
Tue May 5 12:23:50 PDT 2009
Looks good.
Did you rerun your test?
Thanks,
Vladimir
Christian Thalinger wrote:
> On Tue, 2009-05-05 at 11:28 -0700, Tom Rodriguez wrote:
>> I think returning TypeInt::INT directly would be preferred.
>> bottom_type() is always virtual and you have to look elsewhere to see
>> what it returns.
>
> Sounds reasonable. Here is a new version of the patch:
>
> http://cr.openjdk.java.net/~twisti/6823354/webrev.04/
>
> -- Christian
>
More information about the hotspot-compiler-dev
mailing list