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

Issue 25875004: Roll Clang 188423:191856. (Closed)

Created:
7 years, 2 months ago by hans
Modified:
7 years, 2 months ago
Reviewers:
Nico
CC:
chromium-reviews, eugenis+clang_chromium.org, glider+clang_chromium.org, dmikurube+clang_chromium.org, ukai+watch_chromium.org
Visibility:
Public.

Description

Roll Clang 188423:191856. Remove libprofile_rt from package.sh, it doesn't exist after LLVM r191835 BUG=290204, 303886 R=thakis@chromium.org TBR=net owner NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=226920

Patch Set 1 #

Total comments: 1

Patch Set 2 : -Wno-unused-const-variable #

Patch Set 3 : Move -Wno-deprecated-declarations to common.gypi #

Patch Set 4 : also don't break sdch/sdch.gyp #

Patch Set 5 : try to remove -fuse-ld=gold from android's ldflags #

Patch Set 6 : Remove the now broken _Unwind_GetIP work-around #

Patch Set 7 : fix SecurityTest.CallocOverflow #

Patch Set 8 : Disable MultiThreadedCertVerifierTest.CancelRequestThenQuit for LSan #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -19 lines) Patch
M base/debug/stack_trace_android.cc View 1 2 3 4 5 1 chunk +0 lines, -10 lines 0 comments Download
M base/security_unittest.cc View 1 2 3 4 5 6 1 chunk +3 lines, -6 lines 0 comments Download
M build/common.gypi View 1 2 3 4 3 chunks +16 lines, -0 lines 0 comments Download
M net/cert/multi_threaded_cert_verifier_unittest.cc View 1 2 3 4 5 6 7 1 chunk +7 lines, -1 line 0 comments Download
M tools/clang/scripts/package.sh View 1 chunk +0 lines, -1 line 0 comments Download
M tools/clang/scripts/update.sh View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
hans
This depends on getting the clean-ups landed first, but I figured I'd upload it in ...
7 years, 2 months ago (2013-10-03 18:29:43 UTC) #1
Nico
https://codereview.chromium.org/25875004/diff/1/sdch/sdch.gyp File sdch/sdch.gyp (right): https://codereview.chromium.org/25875004/diff/1/sdch/sdch.gyp#newcode63 sdch/sdch.gyp:63: [ 'clang==1', { 'cflags': ['-Wno-deprecated-declarations'] } ], I'm guessing ...
7 years, 2 months ago (2013-10-03 18:41:41 UTC) #2
hans
On 2013/10/03 18:41:41, Nico wrote: > https://codereview.chromium.org/25875004/diff/1/sdch/sdch.gyp > File sdch/sdch.gyp (right): > > https://codereview.chromium.org/25875004/diff/1/sdch/sdch.gyp#newcode63 > ...
7 years, 2 months ago (2013-10-03 18:53:27 UTC) #3
Nico
lgtm
7 years, 2 months ago (2013-10-03 18:55:15 UTC) #4
Nico
clang:error: unknown argument: '-fuse-ld=gold' on android. Augh!
7 years, 2 months ago (2013-10-03 18:56:37 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/hans@chromium.org/25875004/33001
7 years, 2 months ago (2013-10-03 21:51:52 UTC) #6
Nico
7 years, 2 months ago (2013-10-04 01:11:44 UTC) #7
Message was sent while issue was closed.
Committed patchset #8 manually as r226920 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698