New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 775171 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac , Fuchsia
Pri: 2
Type: Bug


Sign in to add a comment

Roll clang again (r315618+)

Project Member Reported by h...@chromium.org, Oct 16 2017

Issue description

Tracking bug for the next Clang roll.

We need:

r315618 for  Bug 774973 
A fix to https://llvm.org/PR34944 for Bug 774742
 

Comment 1 by thakis@chromium.org, Oct 16 2017

Blocking: 774209

Comment 2 by r...@chromium.org, Oct 17 2017

Blockedon: 775328

Comment 3 by thakis@chromium.org, Oct 26 2017

Blockedon: 778157
316713 for  issue 778209 

Comment 4 by thakis@chromium.org, Oct 27 2017

r316385 for issue 495204

Comment 5 by thakis@chromium.org, Oct 27 2017

Blocking: 760838
r316775 for  issue 760838  

Comment 6 by thakis@chromium.org, Oct 30 2017

Blocking: 775174

Comment 7 by thakis@chromium.org, Oct 30 2017

r316807 for  issue 775174 

Comment 8 by p...@chromium.org, Oct 30 2017

Blocking: 778087

Comment 9 by p...@chromium.org, Oct 31 2017

Blockedon: 779891

Comment 10 by r...@chromium.org, Oct 31 2017

Blocking: 694670
r317011 for  issue 694670 
778157 for the libLTO.dylib thing ( issue 778157 ). we don't bundle libLTO so not super important, but cleaner.
that should've said r317010, but it's before reid's change in comment 11 anyhow
probably want to hold for  issue 780079 's resolution in case it's easy-ish

Comment 15 by h...@chromium.org, Nov 1 2017

Blocking: 780079
> probably want to hold for  issue 780079 's resolution in case it's easy-ish

Sounds like that's r317041

Comment 16 by h...@chromium.org, Nov 1 2017

Blockedon: 780523

Comment 17 by h...@chromium.org, Nov 2 2017

Blockedon: 780692

Comment 18 by h...@chromium.org, Nov 2 2017

Status: Started (was: Available)
The bots are still red from the warnings yesterday, but let's start building packages to save time.

Trying 317207 here: https://chromium-review.googlesource.com/#/c/chromium/src/+/751802

Comment 19 by h...@chromium.org, Nov 2 2017

Need r317263 to fix tools/dsymutil/X86/alias.test.
Trying that here: https://chromium-review.googlesource.com/#/c/chromium/src/+/752143
Project Member

Comment 20 by bugdroid1@chromium.org, Nov 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c4a8c7b5ff68ad06bdaff91be5170e4da89c3a9f

commit c4a8c7b5ff68ad06bdaff91be5170e4da89c3a9f
Author: Hans Wennborg <hans@chromium.org>
Date: Fri Nov 03 05:28:21 2017

Roll clang 315613:317263.

Bug:  775171 

Change-Id: I5e930bc3368948673c02b2be325a2ad03ae58606
Reviewed-on: https://chromium-review.googlesource.com/752143
Reviewed-by: Reid Kleckner <rnk@chromium.org>
Commit-Queue: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#513697}
[modify] https://crrev.com/c4a8c7b5ff68ad06bdaff91be5170e4da89c3a9f/build/config/compiler/BUILD.gn
[modify] https://crrev.com/c4a8c7b5ff68ad06bdaff91be5170e4da89c3a9f/tools/clang/scripts/update.py

Comment 21 by h...@chromium.org, Nov 7 2017

Status: Fixed (was: Started)
This appears to be sticking.

The only real fall-out I've seen is  http://crbug.com/781873 , which will get fixed in the next roll.
FYI, is a size regression that was filed as  bug 781272 .

Sign in to add a comment