RFR: 8307160: [REDO] Enable the permissive- flag on the Microsoft Visual C compiler [v7]
Phil Race
prr at openjdk.org
Tue Oct 10 22:38:08 UTC 2023
On Tue, 10 Oct 2023 03:44:27 GMT, Julian Waters <jwaters at openjdk.org> wrote:
>> We should set the -permissive- flag for the Microsoft Visual C compiler, as was requested by the now backed out [JDK-8241499](https://bugs.openjdk.org/browse/JDK-8241499). Doing so makes the Visual C compiler much less accepting of ill formed code, which will improve code quality on Windows in the future.
>
> Julian Waters has updated the pull request incrementally with five additional commits since the last revision:
>
> - Revert sspi.cpp
> - Revert NativeCreds.c
> - Revert allocation.cpp
> - Revert symbolengine.cpp
> - Revert os_windows.cpp
make/autoconf/flags-cflags.m4 line 566:
> 564: # compilation of all source files regardless of the active code page on Windows.
> 565: TOOLCHAIN_CFLAGS_JVM="-nologo -MD -Zc:preprocessor -Zc:strictStrings -Zc:inline -permissive- -utf-8 -MP"
> 566: TOOLCHAIN_CFLAGS_JDK="-nologo -MD -Zc:preprocessor -Zc:strictStrings -Zc:inline -permissive- -utf-8 -Zc:wchar_t-"
Now you are splitting it, surely you can't include this option until the very last PR ?
I don't see those other PRs.
Also the bug cat probably should now be "client-libs" .. I will update it .. and you'll need to create new
linked bugs for the other PRs.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15096#discussion_r1353502377
More information about the build-dev
mailing list