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