RFR: 8333566: Remove unused methods

Amit Kumar amitkumar at openjdk.org
Thu Jun 6 19:23:46 UTC 2024


On Tue, 4 Jun 2024 20:51:52 GMT, Cesar Soares Lucas <cslucas at openjdk.org> wrote:

> Please, consider this patch to remove unused methods from the code base. To the best of my knowledge, these methods are only defined but never used.
> 
> Here is a list with names of delete methods: https://gist.github.com/JohnTortugo/fccc29781a1b584c03162aa4e160e874
> 
> Tested with Linux x86_64 tier1-4, GHA, and only cross building to other platforms.

src/hotspot/cpu/s390/vm_version_s390.hpp line 516:

> 514:   static void set_has_CompareTrap()               { _features[0] |= GnrlInstrExtFacilityMask; }
> 515:   static void set_has_RelativeLoadStore()         { _features[0] |= GnrlInstrExtFacilityMask; }
> 516:   static void set_has_ProcessorAssist()           { _features[0] |= ProcessorAssistMask; }

This looks incorrect; there exist a second definition below;

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/19550#discussion_r1630102121


More information about the shenandoah-dev mailing list