|
|
Created:
8 years ago by Sam Clegg Modified:
8 years ago CC:
chromium-reviews Base URL:
svn://svn.chromium.org/chrome/trunk/src Visibility:
Public. |
DescriptionUpdate 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 : #Messages
Total messages: 28 (0 generated)
+thestig
Can you just flip the flag for this condition near line 540 instead?
On 2012/12/05 23:17:56, Lei Zhang wrote: > Can you just flip the flag for this condition near line 540 instead? The linux_use_gold_flags controls more than just the --threads arg. It also appears to control the --icf arg, which I didn't want to mess with. Do we want a new flag for "gold_use_threads"? I makes sense to me as this needs to be disabled for chromeos and also will be needed for arm/linux builds too.
Ok, thanks for pointing that out. lgtm
On 2012/12/05 23:26:01, Sam Clegg wrote: > Do we want a new flag for "gold_use_threads"? I makes sense to me as this needs > to be disabled for chromeos and also will be needed for arm/linux builds too. I'm really hoping this will be short lived and the gold bug will be fixed soon. So let's not add more flags.
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
Retried try job too often on linux_chromeos for step(s) unit_tests
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
Retried try job too often on linux_chromeos for step(s) unit_tests
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
Retried try job too often on linux_chromeos for step(s) unit_tests
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
lgtm
Retried try job too often on linux_chromeos for step(s) unit_tests
On 2012/12/06 21:51:24, I haz the power (commit-bot) wrote: > Retried try job too often on linux_chromeos for step(s) unit_tests Any idea why this change can't get past the CQ? I can't believe this change effects the chromeos unit tests in any way can it?
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
Retried try job too often on linux_chromeos for step(s) unit_tests
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
Retried try job too often on linux_chromeos for step(s) unit_tests
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
Retried try job too often on linux_chromeos for step(s) unit_tests
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
Retried try job too often on linux_chromeos for step(s) unit_tests
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
Retried try job too often on linux_chromeos for step(s) unit_tests
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sbc@chromium.org/11444012/2001
Message was sent while issue was closed.
Change committed as 171917 |