[jdk8u-dev] RFR: 8297329: [8u] hotspot needs to recognise VS2019 [v2]
Alexey Pavlyutkin
duke at openjdk.org
Mon Nov 21 12:34:40 UTC 2022
> Hi!
>
> Please review another portion of changes to add MSVS2019 support to jdk-8. This one adds recognition of MSVS2019 compiler/linker to the hotspot build scripts and apply build rules accordingly. Before the patch the scripts didn't not know how to link precompiled headers and builds failed with the error
>
> ad_x86_64_pipeline.obj : error LNK2011: precompiled object not linked in; image may not run
> jvm.dll : fatal error LNK1120: 1 unresolved externals
>
>
> Verification: MSVS2019 (32/64-bits, release/debug) builds now fail with
>
> awt_DCHolder.cpp
> C:\PROGRA~2\MICROS~2\2019\COMMUN~1\VC\Tools\MSVC\14.29.30133\include\cstdlib(46): error C2039: 'Do_Not_Use_calloc_Use_safe_Calloc_Instead': is not a member of '`global namespace''
> C:\PROGRA~2\MICROS~2\2019\COMMUN~1\VC\Tools\MSVC\14.29.30133\include\cstdlib(46): error C2873: 'Do_Not_Use_calloc_Use_safe_Calloc_Instead': symbol cannot be used in a using-declaration
> C:\PROGRA~2\MICROS~2\2019\COMMUN~1\VC\Tools\MSVC\14.29.30133\include\cstdlib(52): error C2039: 'Do_Not_Use_malloc_Use_safe_Malloc_Instead': is not a member of '`global namespace''
> C:\PROGRA~2\MICROS~2\2019\COMMUN~1\VC\Tools\MSVC\14.29.30133\include\cstdlib(52): error C2873: 'Do_Not_Use_malloc_Use_safe_Malloc_Instead': symbol cannot be used in a using-declaration
> C:\PROGRA~2\MICROS~2\2019\COMMUN~1\VC\Tools\MSVC\14.29.30133\include\cstdlib(58): error C2039: 'Do_Not_Use_realloc_Use_safe_Realloc_Instead': is not a member of '`global namespace''
> slowdebug/jdk/objs/libawt/awt_DCHolder.obj] Error 2
> make[2]: *** Waiting for unfinished jobs....
>
> that will be fixed by backporting of [JDK-8241087](https://bugs.openjdk.org/browse/JDK-8241087)
>
> Regression: MSVS2017 (32/64-bits, release/debug) build - Ok
Alexey Pavlyutkin has updated the pull request incrementally with one additional commit since the last revision:
fix line alignment to be compliant with the baseline
-------------
Changes:
- all: https://git.openjdk.org/jdk8u-dev/pull/187/files
- new: https://git.openjdk.org/jdk8u-dev/pull/187/files/38050b1f..a7a11fdc
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=187&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=187&range=00-01
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk8u-dev/pull/187.diff
Fetch: git fetch https://git.openjdk.org/jdk8u-dev pull/187/head:pull/187
PR: https://git.openjdk.org/jdk8u-dev/pull/187
More information about the jdk8u-dev
mailing list