Unused code in prims/methodComparator

David Holmes david.holmes at oracle.com
Fri Feb 5 02:05:20 UTC 2016


HI Thomas,

Ultimately this is part of JVMTI so adding the serviceability folk.

On 20/01/2016 6:31 PM, Thomas Stüfe wrote:
> Hi all,
>
> when investigating a warning about unchecked realloc() usage in
> methodComparator.hpp, I found that the offending function
> (BciMap::store_fragment_location) is not callable anymore.
>
> BciMap::store_fragment_location is called
> by MethodComparator::methods_switchable, which is not called anywhere.
>
> Is there any reason we should keep this?

Certainly seems to be dead code. Unclear when/why redefinition dropped 
the notion of "switchable".

Thanks,
David

> Regards, Thomas
>


More information about the hotspot-runtime-dev mailing list