Update tools. This is a reland 9d1f96606cd7af3b8d441fbc5e8d2e69da37c506, which should hopefuly be fine after afd1cd959e75eb8f62fcfc6b1c1124537a09b2f9. Though I've also gone ahead and gotten the latest versions of things. (android_tools and clang updated.) In particular, get the new NDK. Unfortunately, the new clang picks up an unfortunate change for clang-cl that we now must work around. http://llvm.org/viewvc/llvm-project?view=revision&revision=319116 Bug: 109 Change-Id: If19b09c585957fefaffa8c3197a50189402a555a Reviewed-on: https://boringssl-review.googlesource.com/25025 Reviewed-by: Steven Valdez <svaldez@google.com> Commit-Queue: David Benjamin <davidben@google.com> CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>
diff --git a/CMakeLists.txt b/CMakeLists.txt index 7862307..46a8465 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt
@@ -48,17 +48,18 @@ if(CMAKE_COMPILER_IS_GNUCXX OR CLANG) # Note clang-cl is odd and sets both CLANG and MSVC. We base our configuration - # primarily on our normal Clang one because the MSVC one is mostly - # suppressions for an overaggressive -Wall. - set(C_CXX_FLAGS "-Wall -Werror -Wformat=2 -Wsign-compare -Wmissing-field-initializers -Wwrite-strings") + # primarily on our normal Clang one. + set(C_CXX_FLAGS "-Werror -Wformat=2 -Wsign-compare -Wmissing-field-initializers -Wwrite-strings") if(MSVC) - # clang-cl sets different default warnings than clang. - set(C_CXX_FLAGS "${C_CXX_FLAGS} -Wno-unused-parameter -fmsc-version=1900") + # clang-cl sets different default warnings than clang. It also treats -Wall + # as -Weverything, to match MSVC. Instead -W3 is the alias for -Wall. + # See http://llvm.org/viewvc/llvm-project?view=revision&revision=319116 + set(C_CXX_FLAGS "${C_CXX_FLAGS} -W3 -Wno-unused-parameter -fmsc-version=1900") # googletest suppresses warning C4996 via a pragma, but clang-cl does not # honor it. Suppress it here to compensate. See https://crbug.com/772117. set(C_CXX_FLAGS "${C_CXX_FLAGS} -Wno-deprecated-declarations") else() - set(C_CXX_FLAGS "${C_CXX_FLAGS} -ggdb -fvisibility=hidden -fno-common") + set(C_CXX_FLAGS "${C_CXX_FLAGS} -Wall -ggdb -fvisibility=hidden -fno-common") endif() if(CLANG)
diff --git a/util/bot/DEPS b/util/bot/DEPS index a4c3a7f..b58d104 100644 --- a/util/bot/DEPS +++ b/util/bot/DEPS
@@ -22,7 +22,7 @@ deps = { 'boringssl/util/bot/android_tools': { - 'url': Var('chromium_git') + '/android_tools.git' + '@' + 'a2e9bc7c1b41d983577907df51d339fb1e0fd02f', + 'url': Var('chromium_git') + '/android_tools.git' + '@' + 'c9f9bbf0a6c862fbef6115e80e8617093cd58e6b', 'condition': 'checkout_android', }, @@ -30,7 +30,7 @@ Var('chromium_git') + '/external/gyp.git' + '@' + 'd61a9397e668fa9843c4aa7da9e79460fe590bfb', 'boringssl/util/bot/libFuzzer': { - 'url': Var('chromium_git') + '/chromium/llvm-project/compiler-rt/lib/fuzzer.git' + '@' + 'a00e8070bea627f21df4a7eb1d58083f1bcfbba1', + 'url': Var('chromium_git') + '/chromium/llvm-project/compiler-rt/lib/fuzzer.git' + '@' + 'ba2c1cd6f87accb32b5dbce297387c56a2e53a2f', 'condition': 'checkout_fuzzer', }, }
diff --git a/util/bot/update_clang.py b/util/bot/update_clang.py index ffe7880..dceb585 100644 --- a/util/bot/update_clang.py +++ b/util/bot/update_clang.py
@@ -19,8 +19,8 @@ # CLANG_REVISION and CLANG_SUB_REVISION determine the build of clang # to use. These should be synced with tools/clang/scripts/update.py in # Chromium. -CLANG_REVISION = '318667' -CLANG_SUB_REVISION=1 +CLANG_REVISION = '321529' +CLANG_SUB_REVISION=2 PACKAGE_VERSION = "%s-%s" % (CLANG_REVISION, CLANG_SUB_REVISION)
diff --git a/util/bot/vs_toolchain.py b/util/bot/vs_toolchain.py index 64da79a..71b59c8 100644 --- a/util/bot/vs_toolchain.py +++ b/util/bot/vs_toolchain.py
@@ -87,8 +87,9 @@ # Update 3 final with 10.0.15063.468 SDK and no vctip.exe. return ['f53e4598951162bad6330f7a167486c7ae5db1e5'] if env_version == '2017': - # VS 2017 Update 3.2 with 10.0.15063.468 SDK and patched setenv.cmd. - return ['a9e1098bba66d2acccc377d5ee81265910f29272'] + # VS 2017 Update 3.2 with 10.0.15063.468 SDK, patched setenv.cmd, and + # 10.0.16299.15 debuggers. + return ['1180cb75833ea365097e279efb2d5d7a42dee4b0'] raise Exception('Unsupported VS version %s' % env_version)