RFR: 8377726: Remove unused macros in register.hpp
Manuel Hässig
mhaessig at openjdk.org
Wed Feb 18 13:37:32 UTC 2026
On Wed, 18 Feb 2026 10:55:40 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:
>> Please review this change to remove some unused (or nearly unused) macros in
>> share/asm/register.hpp. This change also simplifies the zero definition of
>> noreg to render one of the macros no longer used.
>>
>> Testing: mach5 tier1 and GHA Sanity Tests.
>> The GHA testing failed to build on Alpine Linux because of the unrelated
>> https://bugs.openjdk.org/browse/JDK-8377707
>
> src/hotspot/share/asm/register.hpp line 52:
>
>> 50: // Macros to help define all kinds of registers
>> 51:
>> 52: #ifdef USE_POINTERS_TO_REGISTER_IMPL_ARRAY
>
> Why `ifndef` -> `ifdef`?
Because only the `#else` branch of the original code is left.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29685#discussion_r2822382014
More information about the hotspot-dev
mailing list