Use llvm-ar when building with Clang.

Use llvm-ar when building with Clang.

llvm-ar is faster and is capable of handling
LLVM bitcode files without a need for a Gold plugin.

BUG=660216, 607968
Review-Url: https://codereview.chromium.org/2766333002
Cr-Commit-Position: refs/heads/master@{#460129}
Committed: https://chromium.googlesource.com/chromium/src/+/8968cd5170f4a22b1c508ffa3bc240b346c44b1b

[krasin1, 2017-03-27 17:03:34]

The CQ bit was checked by krasin@chromium.org to run a CQ dry run

[krasin1, 2017-03-27 17:03:49]

krasin@chromium.org changed reviewers:
+ thakis@chromium.org

[commit-bot: I haz the power, 2017-03-27 17:04:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Nico, 2017-03-27 17:10:39]

Lgtm, but let's wait a few hours to make sure the roll sticks. Thanks!

https://codereview.chromium.org/2766333002/diff/1/build/config/compiler/BUILD.gn
File build/config/compiler/BUILD.gn (right):

https://codereview.chromium.org/2766333002/diff/1/build/config/compiler/BUILD...
build/config/compiler/BUILD.gn:305: cflags += [ "-pipe" ]  # Use pipes for
communicating between sub-processes. Faster.
Nit: put comment above += line (yes, not your code)

[krasin1, 2017-03-27 17:51:23]

Sure, I will wait.

https://codereview.chromium.org/2766333002/diff/1/build/config/compiler/BUILD.gn
File build/config/compiler/BUILD.gn (right):

https://codereview.chromium.org/2766333002/diff/1/build/config/compiler/BUILD...
build/config/compiler/BUILD.gn:305: cflags += [ "-pipe" ]  # Use pipes for
communicating between sub-processes. Faster.
On 2017/03/27 17:10:39, Nico wrote:
> Nit: put comment above += line (yes, not your code)

Done.

[krasin1, 2017-03-27 17:51:32]

The CQ bit was checked by krasin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-27 17:52:20]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-27 19:13:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-27 19:13:25]

Dry run: This issue passed the CQ dry run.

[krasin1, 2017-03-28 15:59:38]

The CQ bit was checked by krasin@chromium.org

[krasin1, 2017-03-28 15:59:39]

The patchset sent to the CQ was uploaded after l-g-t-m from thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/2766333002/#ps20001
(title: "Move a comment")

[commit-bot: I haz the power, 2017-03-28 16:00:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-28 16:46:20]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490716778896180,
"parent_rev": "89a520a13e387fc2331943aeb451b16026e9e00e", "commit_rev":
"8968cd5170f4a22b1c508ffa3bc240b346c44b1b"}

[commit-bot: I haz the power, 2017-03-28 16:47:06]

Description was changed from

==========
Use llvm-ar when building with Clang.

llvm-ar is faster and is capable of handling
LLVM bitcode files without a need for a Gold plugin.

BUG=660216,607968
==========

to

==========
Use llvm-ar when building with Clang.

llvm-ar is faster and is capable of handling
LLVM bitcode files without a need for a Gold plugin.

BUG=660216,607968

Review-Url: https://codereview.chromium.org/2766333002
Cr-Commit-Position: refs/heads/master@{#460129}
Committed:
https://chromium.googlesource.com/chromium/src/+/8968cd5170f4a22b1c508ffa3bc2...
==========

[commit-bot: I haz the power, 2017-03-28 16:47:07]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/8968cd5170f4a22b1c508ffa3bc2...