[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