0

build: Remove mac/32-bit x86 conditional from use_v8_context_snapshot

On a mac host, current_cpu should never be x86 when building blink
I think. (Except when cross-building win/x86, but in that case the
target_os=="win" && host_os!="win" further down is already false.)

Hopefully no behavior change.

Bug: none
Change-Id: Ib977564e2f485bbf83a2463a5e1024f0522046a4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4438109
Reviewed-by: Leonard Grey <lgrey@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Auto-Submit: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1131920}
This commit is contained in:
Nico Weber
2023-04-18 15:15:33 +00:00
committed by Chromium LUCI CQ
parent b2bd8ed692
commit 665639a5cd

@ -17,7 +17,6 @@ declare_args() {
# Disable in mac and win cross builds since building Blink twice is slow.
use_v8_context_snapshot =
!is_chromeos && !is_android && !is_castos && !is_fuchsia &&
!(host_os == "mac" && current_cpu == "x86") &&
# Android may build for both 64 bit and 32bit. When this happens, the
# v8_target_cpu will not equal the target_cpu (for example,
# v8_target_cpu == "arm" but target_os == "arm64").