RFR: 8219860: Cleanup ClassFileParser::parse_linenumber_table

Claes Redestad claes.redestad at oracle.com
Fri Mar 8 10:45:27 UTC 2019


Hi Robbin,

On 2019-03-08 11:34, Robbin Ehn wrote:
> Hi Claes.
> 
> Removes 4 times more than it adds, great!
> It seems very reasonably.

thanks!

> 
> Very minor nit, I would remove TRAPS from get_u1(TRAPS) if it's not 
> needed, and I can't see that it is. Keep it if you think it makes some 
> sense.

I think it might be necessary for the exception handling down that
path, which is admittedly a bit hairy. Either way I've just moved this
pre-existing method to help with inlining and would like to keep this
cleanup from spiraling off in yet another direction. :)

/Claes


More information about the hotspot-runtime-dev mailing list