New issue
Advanced search Search tips

Issue 764744 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug


Sign in to add a comment

Roll Clang again

Project Member Reported by h...@chromium.org, Sep 13 2017

Issue description

We'd like at least r313108, which picks up rnk's instcombine change
 

Comment 1 by h...@chromium.org, Sep 13 2017

Blocking: 761633

Comment 2 by h...@chromium.org, Sep 13 2017

Blocking: 753736
Status: Started (was: Assigned)
Trying r313138, https://chromium-review.googlesource.com/665341

Comment 4 by r...@chromium.org, Sep 13 2017

I was going to say let's wait for Adrian's https://reviews.llvm.org/D37715 which should fix  issue 755841 , but rolls are relatively cheap.

Comment 5 by h...@chromium.org, Sep 13 2017

Cool! I didn't realize that one was almost fixed.

I also don't think we should wait for it, but aim to roll twice.

Comment 6 by h...@chromium.org, Sep 13 2017

Blocking: 756175
More practice doing clang rolls for me! I'm all for it.

Comment 8 by h...@chromium.org, Sep 13 2017

Blockedon: 764969

Comment 9 by h...@chromium.org, Sep 13 2017

We need r313213 for the  Issue 764969  fix.

Comment 10 by h...@chromium.org, Sep 14 2017

Blockedon: 765004

Comment 11 by h...@chromium.org, Sep 14 2017

We need r313222 for  Issue 765004 .

Comment 12 by p...@chromium.org, Sep 14 2017

Blocking: 607968
Once we roll past r313078, I'm hoping that we'll be able to switch the default linker on Linux to lld.

Comment 13 by p...@chromium.org, Sep 14 2017

Cc: p...@chromium.org
Project Member

Comment 14 by bugdroid1@chromium.org, Sep 15 2017

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

commit b69b8d3f01c87dde1d3cdf81d765f90d54b9b17d
Author: inglorion <inglorion@chromium.org>
Date: Fri Sep 15 02:23:16 2017

Roll clang 312679:313222.

TBR=thakis

Bug:  764744 
Change-Id: I2841a5041b7a3d1ac14816f183dea93b671e79c7
Reviewed-on: https://chromium-review.googlesource.com/665341
Reviewed-by: Hans Wennborg <hans@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502136}
[modify] https://crrev.com/b69b8d3f01c87dde1d3cdf81d765f90d54b9b17d/build/config/compiler/BUILD.gn
[modify] https://crrev.com/b69b8d3f01c87dde1d3cdf81d765f90d54b9b17d/tools/clang/scripts/update.py

Comment 15 by h...@chromium.org, Sep 15 2017

Status: Fixed (was: Started)
It seems to be sticking \o/

Sign in to add a comment