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

Issue 1229083005: clang/win: Add back /fallback for asan builds for now. (Closed)

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

Description

clang/win: Add back /fallback for asan builds for now. In asan builds, a few translation units still fall back to CL, see http://llvm.org/PR24167 Also update the gn files to pass /WX (warnings-as-errors) in 32-bit non-asan clang builds. BUG=82385 TBR=rnk Committed: https://chromium.googlesource.com/chromium/src/+/28d32d89e5081223fab2982dc97252fd5150e721

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -1 line) Patch
M build/common.gypi View 1 chunk +9 lines, -0 lines 0 comments Download
M build/config/compiler/BUILD.gn View 2 chunks +5 lines, -1 line 0 comments Download

Messages

Total messages: 5 (1 generated)
Nico
5 years, 5 months ago (2015-07-17 19:12:25 UTC) #2
Nico
tbr'ing to hopefully get the win/asan bot back green
5 years, 5 months ago (2015-07-17 20:00:03 UTC) #3
Nico
Committed patchset #1 (id:1) manually as 28d32d89e5081223fab2982dc97252fd5150e721 (presubmit successful).
5 years, 5 months ago (2015-07-17 20:05:25 UTC) #4
Reid Kleckner
5 years, 5 months ago (2015-07-17 20:26:31 UTC) #5
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698