From 4e03b98e43539a08abc0e6504fe3b795f4e7a5fb Mon Sep 17 00:00:00 2001 From: Sergey Chernyshev Date: Thu, 25 Apr 2024 04:23:14 +0200 Subject: [PATCH] tools: fix --v8-non-optimized-debug for v18.x MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Disable SLOW_DCHECKs on v18.x because they don't compile without the patches in https://bugs.chromium.org/p/v8/issues/detail?id=12887 if used in constexpr, which can happen in other floated patches. PR-URL: https://github.com/nodejs/node/pull/50612 Reviewed-By: Michaël Zasso --- graal-nodejs/tools/v8_gypfiles/features.gypi | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/graal-nodejs/tools/v8_gypfiles/features.gypi b/graal-nodejs/tools/v8_gypfiles/features.gypi index 3c94b0bca4c..aa59713bbbe 100644 --- a/graal-nodejs/tools/v8_gypfiles/features.gypi +++ b/graal-nodejs/tools/v8_gypfiles/features.gypi @@ -90,6 +90,11 @@ # Set to 1 to enable DCHECKs in release builds. 'dcheck_always_on%': 0, + # For v18.x, disable SLOW_DCHECKs because they don't compile without + # the patches in https://bugs.chromium.org/p/v8/issues/detail?id=12887 + # if used in constexpr, which can happen in other floated patches. + 'v8_enable_slow_dchecks%': 0, + # Sets -DV8_ENABLE_FUTURE. 'v8_enable_future%': 0,