New issue
Advanced search Search tips
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
link

Issue 775171: Roll clang again (r315618+)

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

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

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

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

Blocking: 775174

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

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

Comment 11 by thakis@chromium.org, Oct 31 2017

Comment 12 by thakis@chromium.org, Oct 31 2017

778157 for the libLTO.dylib thing ( issue 778157 ). we don't bundle libLTO so not super important, but cleaner.

Comment 13 by thakis@chromium.org, Oct 31 2017

that should've said r317010, but it's before reid's change in comment 11 anyhow

Comment 14 by thakis@chromium.org, Oct 31 2017

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

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

Project Member
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.

Comment 22 by thestig@chromium.org, Dec 20 2017

FYI, is a size regression that was filed as  bug 781272 .

Sign in to add a comment