RFR: JDK-8253239: Disable VS warning C4307 [v2]
Matthias Baesken
mbaesken at openjdk.java.net
Fri Sep 18 10:05:10 UTC 2020
> hello, this fixes the build when using VS2017. VS2019 does not have the issue as far as I know.
> failure was
>
>> ./test/hotspot/gtest/utilities/test_align.cpp(96): error C2220: warning treated as error - no 'object' file generated
>> ./test/hotspot/gtest/utilities/test_align.cpp(156): note: see reference to function template instantiation 'void
>> static_test_alignments<T,A>(void)' being compiled with
>> [
>> T=int64_t,
>> A=uint8_t
>> ]
>> ./test/hotspot/gtest/utilities/test_align.cpp(162): note: see reference to function template instantiation 'void
>> test_alignments<int64_t,uint8_t>(void)' being compiled ./test/hotspot/gtest/utilities/test_align.cpp(96): warning
>> C4307: '+': integral constant overflow
>>
>>
>
> This might be related to an issue fixed at least in some versions of VS2019 , that is discussed here :
>
> https://developercommunity.visualstudio.com/content/problem/211134/unsigned-integer-overflows-in-constexpr-functionsa.html
>
> https://bugs.openjdk.java.net/browse/JDK-8253239
Matthias Baesken has refreshed the contents of this pull request, and previous commits have been removed. The
incremental views will show differences compared to the previous content of the PR. The pull request contains one new
commit since the last revision:
JDK-8253239
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/237/files
- new: https://git.openjdk.java.net/jdk/pull/237/files/0d1d38b3..4e82edcb
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=237&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=237&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.java.net/jdk/pull/237.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/237/head:pull/237
PR: https://git.openjdk.java.net/jdk/pull/237
More information about the build-dev
mailing list