RFR: 8339783: Implement JEP 479: Remove the Windows 32-bit x86 Port [v13]
Magnus Ihse Bursie
ihse at openjdk.org
Wed Oct 30 13:37:18 UTC 2024
On Wed, 30 Oct 2024 12:11:26 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Magnus Ihse Bursie has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix 32/64-bit confusion in comment in VirtualMachineImpl.c
>
> make/hotspot/gensrc/GensrcAdlc.gmk line 50:
>
>> 48: ADLC_CFLAGS := -nologo -EHsc
>> 49: ADLC_CFLAGS_WARNINGS := -W3 -D_CRT_SECURE_NO_WARNINGS \
>> 50: -D_CRT_DECLARE_NONSTDC_NAMES -D_CRT_NONSTDC_NO_WARNINGS
>
> Not clear why do we need these new warnings? I don't right away see anything in ADLC that needs it.
David Holmes [pointed out](https://github.com/openjdk/jdk/pull/21744#discussion_r1820429621) a chunk of old Windows definitions in `adlc.hpp`. I removed it, including the `_strdpup` define, to align with how the rest of Hotspot handles this peculiarity in Visual Studio, but that required adding the two special defines.
That change is arguably outside the scope of this PR. If you object to it, I can revert it and we'll handle that cleanup separately. It's sometimes hard to know where to stop when you start pulling on strings in old bad code and piece after piece of old legacy junk unravels.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21744#discussion_r1822651205
More information about the build-dev
mailing list