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

Issue 604993 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug


Sign in to add a comment

Roll clang once more

Reported by sigbjo...@opera.com, Apr 20 2016

Issue description

Placeholder bug for the next clang roll, previous one being  https://crbug.com/601774 
 

Comment 1 by sigbjo...@opera.com, Apr 20 2016

Blocking: 604463

Comment 2 by sigbjo...@opera.com, Apr 20 2016

Blocking: 604476

Comment 3 by xlai@chromium.org, Apr 20 2016

Blocking: 605286

Comment 4 by thakis@chromium.org, Apr 21 2016

Blocking: 604271

Comment 5 by thakis@chromium.org, Apr 25 2016

Blocking: 605570
Project Member

Comment 6 by bugdroid1@chromium.org, Apr 26 2016

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

commit 8aa292dd634877fc41e5844af23d1dee28694f41
Author: thakis <thakis@chromium.org>
Date: Tue Apr 26 00:24:36 2016

roll clang 266460:267383

Also switch to gnuwin-3, which is identical to gnuwin-1
(which is a zip of all binaries from GnuWin32 needed to
run llvm regression tests), except that rm.exe, mv.exe
and find.exe have been replaced with rm.exe, mv.exe,
and find.exe from msys (and three msys dlls needed
to run them have been added).  This new rm.exe is
able to delete paths longer than 260 characters, needed
for clang's Modules/dependency-dump.m test, and the new
mv.exe is needed for ExecutionEngine/OrcMCJIT/load-object-a.ll
which moves long paths around with `find.exe -exec mv.exe`

BUG= 604993 , 603364 

Review URL: https://codereview.chromium.org/1917853002

Cr-Commit-Position: refs/heads/master@{#389632}

[modify] https://crrev.com/8aa292dd634877fc41e5844af23d1dee28694f41/build/common.gypi
[modify] https://crrev.com/8aa292dd634877fc41e5844af23d1dee28694f41/build/config/compiler/BUILD.gn
[modify] https://crrev.com/8aa292dd634877fc41e5844af23d1dee28694f41/tools/clang/scripts/update.py

Comment 7 by krasin@chromium.org, Apr 27 2016

Cc: kcc@chromium.org p...@chromium.org krasin@chromium.org
Hans, Nico,

we would like to have r267784 rolled into Chrome: http://reviews.llvm.org/rL267784

It should speed up LTO devirtualization, see https://crbug.com/601822.
I realize that you have just rolled Clang two days ago, and might not be willing to do it so quickly again. Shall I open a bug for the next roll then?

Comment 8 by h...@chromium.org, Apr 27 2016

Owner: thakis@chromium.org
Status: Fixed (was: Untriaged)
Optimistically marking this one fixed since the roll has been in for almost two days.

I've filed  crbug.com/607312  for the next one.

Comment 9 by krasin@chromium.org, Apr 27 2016

Thank you, Hans!

Sign in to add a comment