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

Issue 936013002: Use the version of clang shipped with Xcode for official build (Closed)

Created:
5 years, 10 months ago by sdefresne
Modified:
5 years, 10 months ago
Reviewers:
stuartmorgan, Nico
CC:
chromium-reviews, Olivier
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Use the version of clang shipped with Xcode for official build Instead of checking which generator is used to decide whether to use ToT clang or Xcode clang, make the decision based on the buildtype instead. Use the version of clang shipped with Xcode for official build, and leave the default selected for developer builds (default to ToT clang). BUG=459148 Committed: https://crrev.com/3ea2d12051d9922ad4834c2efd19adfa1fe71cf1 Cr-Commit-Position: refs/heads/master@{#317627}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -1 line) Patch
M build/common.gypi View 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 9 (2 generated)
sdefresne
stuartmorgan: this should give us better warnings on the bots without introducing changes to the ...
5 years, 10 months ago (2015-02-18 10:58:39 UTC) #2
sdefresne
Ping?
5 years, 10 months ago (2015-02-23 10:24:02 UTC) #3
Nico
On 2015/02/23 10:24:02, sdefresne wrote: > Ping? Lgtm I think this is great, but I ...
5 years, 10 months ago (2015-02-23 16:55:43 UTC) #4
stuartmorgan
LGTM. (I'm actually just really behind on reviews, and needed to think through the implications ...
5 years, 10 months ago (2015-02-23 17:48:24 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/936013002/1
5 years, 10 months ago (2015-02-23 17:56:32 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 10 months ago (2015-02-23 19:48:59 UTC) #8
commit-bot: I haz the power
5 years, 10 months ago (2015-02-23 19:49:30 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/3ea2d12051d9922ad4834c2efd19adfa1fe71cf1
Cr-Commit-Position: refs/heads/master@{#317627}

Powered by Google App Engine
This is Rietveld 408576698