On Fri, 18 Sep 2020 08:22:56 GMT, Matthias Baesken <mbaes...@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 <mbaes...@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
