Commit db487ec8 authored by calamity's avatar calamity Committed by Chromium LUCI CQ

Revert "[Lacros] Use cfi and thin_lto for Lacros official build"

This reverts commit efe87dc0.

Reason for revert: Suspected of causing lacros-chrome bot to fail
https://ci.chromium.org/p/chrome/builders/ci/lacros-chrome/4176

Original change's description:
> [Lacros] Use cfi and thin_lto for Lacros official build
>
> CFI and ThinLTO were disabled on the simple chrome workflow
> because they increase build time quite a bit.
> But for official build, then should be enabled.
>
> Bug: 1157567
> Change-Id: I33a9eac37221f31da9a6725b19992d5daded72dc
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2586085
> Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
> Reviewed-by: Sven Zheng <svenzheng@chromium.org>
> Commit-Queue: Sven Zheng <svenzheng@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#835956}

TBR=liaoyuke@chromium.org,svenzheng@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com

Change-Id: I666df6aa212da4c97f3679af5a8f77b889d6e9ba
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1157567
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2586169Reviewed-by: default avatarcalamity <calamity@chromium.org>
Commit-Queue: calamity <calamity@chromium.org>
Cr-Commit-Position: refs/heads/master@{#835984}
parent 20704b23
......@@ -1591,7 +1591,7 @@
],
'chromeos_amd64-generic_lacros_official': [
'chromeos_amd64-generic', 'amd64-lacros', 'official', 'minimal_symbols', 'cfi', 'thin_lto',
'chromeos_amd64-generic', 'amd64-lacros', 'official', 'minimal_symbols',
],
'chromeos_amd64-generic_lacros_rel': [
......
......@@ -240,7 +240,6 @@
"cros_host_sysroot": "//build/linux/debian_sid_amd64-sysroot",
"cros_v8_snapshot_sysroot": "//build/linux/debian_sid_amd64-sysroot",
"enable_linux_installer": false,
"is_cfi": true,
"is_chrome_branded": true,
"is_chromeos_device": true,
"is_official_build": true,
......@@ -261,7 +260,6 @@
"use_pangocairo": false,
"use_pulseaudio": false,
"use_system_libsync": false,
"use_thin_lto": true,
"use_v8_context_snapshot": false,
"use_vaapi": false
}
......
......@@ -52,7 +52,6 @@
"cros_host_sysroot": "//build/linux/debian_sid_amd64-sysroot",
"cros_v8_snapshot_sysroot": "//build/linux/debian_sid_amd64-sysroot",
"enable_linux_installer": false,
"is_cfi": true,
"is_chrome_branded": true,
"is_chromeos_device": true,
"is_official_build": true,
......@@ -73,7 +72,6 @@
"use_pangocairo": false,
"use_pulseaudio": false,
"use_system_libsync": false,
"use_thin_lto": true,
"use_v8_context_snapshot": false,
"use_vaapi": false
}
......
......@@ -6,7 +6,6 @@
"cros_host_sysroot": "//build/linux/debian_sid_amd64-sysroot",
"cros_v8_snapshot_sysroot": "//build/linux/debian_sid_amd64-sysroot",
"enable_linux_installer": false,
"is_cfi": true,
"is_chrome_branded": true,
"is_chromeos_device": true,
"is_official_build": true,
......@@ -27,7 +26,6 @@
"use_pangocairo": false,
"use_pulseaudio": false,
"use_system_libsync": false,
"use_thin_lto": true,
"use_v8_context_snapshot": false,
"use_vaapi": false
}
......
......@@ -6,7 +6,6 @@
"cros_host_sysroot": "//build/linux/debian_sid_amd64-sysroot",
"cros_v8_snapshot_sysroot": "//build/linux/debian_sid_amd64-sysroot",
"enable_linux_installer": false,
"is_cfi": true,
"is_chrome_branded": true,
"is_chromeos_device": true,
"is_official_build": true,
......@@ -27,7 +26,6 @@
"use_pangocairo": false,
"use_pulseaudio": false,
"use_system_libsync": false,
"use_thin_lto": true,
"use_v8_context_snapshot": false,
"use_vaapi": false
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment