RFR: 8377658: G1: Cleanup g1ConcurrentMark.cpp for stricter warning flags
Leo Korinth
lkorinth at openjdk.org
Fri Feb 13 09:48:30 UTC 2026
On Thu, 12 Feb 2026 20:31:29 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:
>> Cleanup `g1ConcurrentMark.cpp` and included header files to remove implicit narrowing conversions.
>>
>> For gcc: `-Wconversion -Wno-float-conversion`
>> For clang: `-Wimplicit-int-conversion`
>>
>> `sizeof_auto` is created so that we can reduce the amount of casting and thus make the code more type safe. The normal `sizeof` will return a `size_t` although the size of most stuff can be represented in a `uint8_t`. `sizeof_auto` will return the size in an as small unsigned type as is possible. The result is that expressions that uses `sizeof_auto` can convert to most integral types, and no explicit narrowing cast will be needed.
>
> src/hotspot/share/utilities/globalDefinitions.hpp line 173:
>
>> 171: // Use this instead of sizeof() if you want your expression not to be of type size_t
>> 172: template <typename T>
>> 173: constexpr auto sizeof_auto() {
>
> I would consider extracting this into a separate PR.
Okay
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29701#discussion_r2803275025
More information about the hotspot-dev
mailing list