Integrated: 8303900: Rename BitMap search functions

Kim Barrett kbarrett at openjdk.org
Fri Mar 10 21:20:56 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

This pull request has now been integrated.

Changeset: 21169285
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/2116928528c0554b2ba0171bd7968ab693972804
Stats:     75 lines in 12 files changed: 0 ins; 0 del; 75 mod

8303900: Rename BitMap search functions

Reviewed-by: stefank, aboldtch

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

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


More information about the shenandoah-dev mailing list