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

Issue 11444012: Update gold linker to 2.23.1 (Closed)

Created:
8 years ago by Sam Clegg
Modified:
8 years ago
Reviewers:
Lei Zhang, Nico
CC:
chromium-reviews
Visibility:
Public.

Description

Update gold linker to 2.23.1 This is the second attempt to integrate latest gold version. The first time around there were crashes seen on 32-bit linux hosts. Disabling multi-threaded linking on 32-bit hosts fixes the issue. BUG=161942 NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=171917

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -8 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M build/common.gypi View 1 chunk +12 lines, -7 lines 0 comments Download

Messages

Total messages: 28 (0 generated)
Sam Clegg
8 years ago (2012-12-05 21:12:36 UTC) #1
Sam Clegg
+thestig
8 years ago (2012-12-05 23:07:01 UTC) #2
Lei Zhang
Can you just flip the flag for this condition near line 540 instead?
8 years ago (2012-12-05 23:17:56 UTC) #3
Sam Clegg
On 2012/12/05 23:17:56, Lei Zhang wrote: > Can you just flip the flag for this ...
8 years ago (2012-12-05 23:26:01 UTC) #4
Lei Zhang
Ok, thanks for pointing that out. lgtm
8 years ago (2012-12-05 23:33:56 UTC) #5
Lei Zhang
On 2012/12/05 23:26:01, Sam Clegg wrote: > Do we want a new flag for "gold_use_threads"? ...
8 years ago (2012-12-05 23:35:20 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-06 04:16:03 UTC) #7
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) unit_tests
8 years ago (2012-12-06 05:38:39 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-06 07:52:14 UTC) #9
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) unit_tests
8 years ago (2012-12-06 13:55:21 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-06 16:26:38 UTC) #11
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) unit_tests
8 years ago (2012-12-06 19:40:22 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-06 20:51:25 UTC) #13
Nico
lgtm
8 years ago (2012-12-06 21:39:27 UTC) #14
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) unit_tests
8 years ago (2012-12-06 21:51:24 UTC) #15
Sam Clegg
On 2012/12/06 21:51:24, I haz the power (commit-bot) wrote: > Retried try job too often ...
8 years ago (2012-12-06 21:56:37 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-06 21:59:12 UTC) #17
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) unit_tests
8 years ago (2012-12-06 22:48:59 UTC) #18
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-06 23:45:43 UTC) #19
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) unit_tests
8 years ago (2012-12-07 00:38:05 UTC) #20
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-07 21:29:59 UTC) #21
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) unit_tests
8 years ago (2012-12-07 22:38:30 UTC) #22
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-07 22:45:36 UTC) #23
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) unit_tests
8 years ago (2012-12-08 00:01:50 UTC) #24
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-08 01:23:17 UTC) #25
commit-bot: I haz the power
Retried try job too often on linux_chromeos for step(s) unit_tests
8 years ago (2012-12-08 02:08:37 UTC) #26
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
8 years ago (2012-12-08 02:14:19 UTC) #27
commit-bot: I haz the power
8 years ago (2012-12-08 02:42:55 UTC) #28
Message was sent while issue was closed.
Change committed as 171917

Powered by Google App Engine
This is Rietveld 408576698