Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(187)

Issue 430383002: Use clang as default host compiler when cross-compiling (Closed)

Created:
6 years, 4 months ago by Nico
Modified:
6 years, 4 months ago
Reviewers:
hans
CC:
chromium-reviews, Yaron, Torne, Lei Zhang, awong, jamesr
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Use clang as default host compiler when cross-compiling chrome/android still used system gcc as host compiler, and system gcc is gcc 4.6 on Ubuntu Precise. This is too old. Instead, use clang by default as host compiler (except when building for AOSP, see http://crbug.com/377684 for that). This is controlled by a new gyp variable host_clang, which only has an effect if clang=0. The existing clang gyp variable means "use clang for everything"; if it's not set then host_clang means "use clang for host compiles". If your system gcc is 4.8+ and you don't want to use clang, you can set host_clang=0 to use system gcc for host compiles instead. In host_clang mode, our clang plugins currently aren't used (I figure if you want plugins, you can just use clang=1). clang_warning_flags on the other hand are used, to make sure the build stays warning-free. BUG=360311 R=hans@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=287386

Patch Set 1 #

Patch Set 2 : step 2 #

Patch Set 3 : rebase #

Patch Set 4 : foo #

Patch Set 5 : . #

Patch Set 6 : . #

Patch Set 7 : aosp? #

Patch Set 8 : . #

Total comments: 1

Patch Set 9 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+51 lines, -8 lines) Patch
M build/common.gypi View 1 2 3 4 5 6 7 8 9 chunks +43 lines, -8 lines 0 comments Download
M build/set_clang_warning_flags.gypi View 1 2 3 4 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Nico
6 years, 4 months ago (2014-08-04 16:43:28 UTC) #1
hans
lgtm, Nice! > Use clang as default host compiler when cross-compiling > > chrome/android still ...
6 years, 4 months ago (2014-08-04 16:57:58 UTC) #2
Nico
On 2014/08/04 16:57:58, hans wrote: > lgtm, Nice! > > > Use clang as default ...
6 years, 4 months ago (2014-08-04 17:26:40 UTC) #3
Nico
The CQ bit was checked by thakis@chromium.org
6 years, 4 months ago (2014-08-04 17:44:38 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/430383002/160001
6 years, 4 months ago (2014-08-04 17:46:26 UTC) #5
Nico
6 years, 4 months ago (2014-08-04 19:05:24 UTC) #6
Message was sent while issue was closed.
Committed patchset #9 manually as r287386 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698