RFR: 8367981: Refine CompactHashtable comments
Francesco Andreuzzi
fandreuzzi at openjdk.org
Thu Sep 18 11:19:50 UTC 2025
On Thu, 18 Sep 2025 10:53:25 GMT, Hamlin Li <mli at openjdk.org> wrote:
> Hi,
> Can you help to review this patch?
>
> The comments of CompactHashtable and related classes are out of date, and some comments are unclear, wrong or misleading.
> As the related classes are used in more and more scenarios, it's helpful to update the comments.
>
> Thanks
src/hotspot/share/classfile/compactHashtable.cpp line 195:
> 193: size_t SimpleCompactHashtable::calculate_header_size() {
> 194: // We have 5 fields. Each takes up sizeof(intptr_t). See WriteClosure::do_u4
> 195: constexpr size_t bytes = sizeof(intptr_t) * 5;
The whole method could perhaps become a `constexpr`
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27360#discussion_r2358666947
More information about the hotspot-runtime-dev
mailing list