RFR: 8303900: Rename BitMap search functions

Stefan Karlsson stefank at openjdk.org
Thu Mar 9 18:04:40 UTC 2023


On Thu, 9 Mar 2023 15:34:05 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this renaming of the following functions in BitMap:
> 
> get_next_one_offset => find_first_set_bit
> get_next_zero_offset => find_first_clear_bit
> get_next_one_offset_aligned_right => find_first_set_bit_aligned_right
> 
> Note that ShenandoahMarkBitMap::get_next_one_offset is not being renamed.  For
> some reason that class contains a copy of a sizable chunk of the code from
> some version of BitMap.  (Not sure why it doesn't use an internal BitMapView
> rather than code copying.)
> 
> Testing:
> mach5 tier1-5

Looks good.

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

Marked as reviewed by stefank (Reviewer).

PR: https://git.openjdk.org/jdk/pull/12951


More information about the shenandoah-dev mailing list