On Tue, 10 Jun 2025 09:05:00 GMT, Johan Sjölen <jsjo...@openjdk.org> wrote:
>> src/hotspot/share/utilities/packedTable.hpp line 69: >> >>> 67: // by the supplier (when Supplier::next() returns false the whole >>> array should >>> 68: // be filled). >>> 69: void fill(u1* table, size_t table_length, Supplier &supplier) const; >> >> Let the ampersand hug the type. > > Also seems like this can be a static method? The class constructor calculates the sizes, masks and bitshift for the arguments. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24847#discussion_r2137482532
