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".


> Regards, Thomas

More information about the hotspot-runtime-dev mailing list