From a60a9604064375c54aff535dc4020c595fa1b38a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C3=ABl=20Zasso?= Date: Wed, 7 Jul 2021 13:22:38 +0200 Subject: [PATCH] deps: V8: cherry-pick 895949419186 Original commit message: Add -Wno-string-concatenation to test/cctest:cctest_sources v8/test/cctest/interpreter/test-bytecode-generator.cc contains lots of string arrays with intentional concatenation. Bug: chromium:1114873 Change-Id: Ie9d35c3849b5b0a6d1d01b6ce21fb80a320d8736 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2366829 Commit-Queue: Arthur Eubanks Reviewed-by: Sathya Gunasekaran Cr-Commit-Position: refs/heads/master@{#69625} Refs: https://github.com/v8/v8/commit/8959494191869ff483927916f9f46eb419784f07 PR-URL: https://github.com/nodejs/node/pull/39244 Refs: https://github.com/nodejs/build/issues/2696 Reviewed-By: Richard Lau --- common.gypi | 2 +- deps/v8/test/cctest/BUILD.gn | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/common.gypi b/common.gypi index b622b61086d434..8b81a0a39dd09d 100644 --- a/common.gypi +++ b/common.gypi @@ -36,7 +36,7 @@ # Reset this number to 0 on major V8 upgrades. # Increment by one for each non-official patch applied to deps/v8. - 'v8_embedder_string': '-node.71', + 'v8_embedder_string': '-node.72', ##### V8 defaults for Node.js ##### diff --git a/deps/v8/test/cctest/BUILD.gn b/deps/v8/test/cctest/BUILD.gn index 66e1473ed28d3a..037947c009a556 100644 --- a/deps/v8/test/cctest/BUILD.gn +++ b/deps/v8/test/cctest/BUILD.gn @@ -60,6 +60,12 @@ v8_header_set("cctest_headers") { sources = [ "cctest.h" ] } +config("cctest_sources_config") { + if (is_clang) { + cflags = [ "-Wno-string-concatenation" ] + } +} + v8_source_set("cctest_sources") { testonly = true @@ -398,6 +404,7 @@ v8_source_set("cctest_sources") { "../..:external_config", "../..:internal_config_base", "../..:v8_tracing_config", + ":cctest_sources_config", ] public_deps = [