RFR: JDK-8253239: Disable VS warning C4307 [v3]
Erik Joelsson
erikj at openjdk.java.net
Fri Sep 18 14:09:55 UTC 2020
On Fri, 18 Sep 2020 13:58:57 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
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>
> JDK-8253239: Disable VS warning C4307
>
> Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
Marked as reviewed by erikj (Reviewer).
-------------
PR: https://git.openjdk.java.net/jdk/pull/237
More information about the build-dev
mailing list