[crac] RFR: Compile all the INCLUDE_LD_SO_LIST_DIAGNOSTICS code unconditionally
Jan Kratochvil
jkratochvil at openjdk.org
Tue Oct 31 07:07:58 UTC 2023
On Tue, 31 Oct 2023 06:52:47 GMT, Radim Vansa <rvansa at openjdk.org> wrote:
>> Make INCLUDE_LD_SO_LIST_DIAGNOSTICS an `if` conditional instead of a `#if` one.
>> Fix compilation failures.
>
> src/hotspot/cpu/x86/vm_version_x86.cpp line 856:
>
>> 854: }
>> 855:
>> 856: //#if INCLUDE_LD_SO_LIST_DIAGNOSTICS
>
> Please remove commented out code.
I wanted to indicate this function is used only for `INCLUDE_LD_SO_LIST_DIAGNOSTICS`. I can also put there a full sentence.
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/136#discussion_r1377132708
More information about the crac-dev
mailing list