RFR: 8303900: Rename BitMap search functions
Axel Boldt-Christmas
aboldtch at openjdk.org
Fri Mar 10 12:39:03 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
lgtm
-------------
Marked as reviewed by aboldtch (Committer).
PR: https://git.openjdk.org/jdk/pull/12951
More information about the shenandoah-dev
mailing list