RFR: 8240528: OopMap cleanup

Claes Redestad claes.redestad at oracle.com
Thu Mar 5 11:49:49 UTC 2020


Hi Tobias,

On 2020-03-05 07:24, Tobias Hartmann wrote:
> Hi Claes,
> 
> looks good to me but I think the assert in oopMap.hpp:213 is not needed because the bounds are
> already checked in GrowableArray::at(). Also, it should be index < length() not <=.

thanks for reviewing!

You're right. In my mind I had already removed this redundant assert.
I'll remove it before push.

Thanks!

/Claes


More information about the hotspot-compiler-dev mailing list