Request for reviews (L): 6823354: Add intrinsics for{Integer,Long}.{numberOfLeadingZeros,numberOfTrailingZeros}()

Tom Rodriguez Thomas.Rodriguez at Sun.COM
Tue May 5 13:01:15 PDT 2009


looks good.

tom

On May 5, 2009, at 12:17 PM, 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