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