Integrated: JDK-8253239: Disable VS warning C4307

Matthias Baesken mbaesken at openjdk.java.net
Fri Sep 18 14:09:56 UTC 2020


On Fri, 18 Sep 2020 08:22:56 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: 52c28b86
Author:    Matthias Baesken <mbaesken at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/52c28b86
Stats:     4 lines in 1 file changed: 0 ins; 4 del; 0 mod

8253239: Disable VS warning C4307

Reviewed-by: mdoerr, erikj

-------------

PR: https://git.openjdk.java.net/jdk/pull/237



More information about the build-dev mailing list