Roll Clang 223108:231191

Roll Clang 223108:231191

Also introduce a build number in the package name, stop versioning
the Blink GC plugin, and remove repackage.sh.

BUG=444089
NOPRESUBMIT=true

Committed: https://crrev.com/832e529746438a592d8c0a7edd76a8eb9290c8c3
Cr-Commit-Position: refs/heads/master@{#319215}

[hans, 2015-03-04 05:53:15]

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

[hans, 2015-03-04 05:53:16]

This is the same as previous roll attempt:
https://codereview.chromium.org/975503002/, but it targets 231191 to pick up the
fix mentioned in https://code.google.com/p/chromium/issues/detail?id=444089#c121

The previous roll was reverted due to a problem in V8, which was fixed in
https://codereview.chromium.org/979633002/. We need to wait until that fix has
rolled into Chromium.

[hans, 2015-03-04 05:54:24]

Forgot to mention, the fix mentioned in
https://code.google.com/p/chromium/issues/detail?id=444089#c121 was actually in
Clang r231156, but that revision has a test error on Darwin, which got fixed in
r231191.

[Nico, 2015-03-04 06:20:09]

lgtm

[hans, 2015-03-04 17:34:06]

Still waiting for V8 roll :/

Tryjob status:
- android_rel_tests_recipe fails due to lack of V8 roll
- linux_chromium_trusty32_rel bot is broken
- linux_chromium_chromeos_asan_rel_ng broken; see whitespace change at
https://codereview.chromium.org/405473002

[hans, 2015-03-04 21:12:19]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2015-03-04 21:13:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/980523002/1

[commit-bot: I haz the power, 2015-03-04 21:13:52]

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

[commit-bot: I haz the power, 2015-03-04 21:13:53]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
in order for the CQ to process them

[hans, 2015-03-04 21:29:56]

The CQ bit was checked by hans@chromium.org

[hans, 2015-03-04 21:29:56]

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/980523002/#ps20001 (title:
"Rebase on top of recent update.sh change")

[commit-bot: I haz the power, 2015-03-04 21:31:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/980523002/20001

[commit-bot: I haz the power, 2015-03-04 21:53:05]

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

[commit-bot: I haz the power, 2015-03-04 21:53:06]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[hans, 2015-03-04 21:53:40]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2015-03-04 21:54:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/980523002/20001

[hans, 2015-03-05 00:03:58]

The CQ bit was unchecked by hans@chromium.org

[hans, 2015-03-05 00:04:22]

I've unchecked the cq box until the goma issues are resolved.

[hans, 2015-03-05 01:32:31]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2015-03-05 01:33:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/980523002/20001

[commit-bot: I haz the power, 2015-03-05 04:02:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-05 04:03:26]

Patchset 2 (id:??) landed as
https://crrev.com/832e529746438a592d8c0a7edd76a8eb9290c8c3
Cr-Commit-Position: refs/heads/master@{#319215}

[Finnur, 2015-03-05 09:54:21]

This Cl caused the official Linux x64 bot to go red due to the sizes check.
https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/78934

[Nico, 2015-03-05 14:34:15]

It made chrome over 1MB smaller, it's OK if it adds ten kB to nacl_helper
:-)

That bot really should call eu-strip before printing the size BTW. The size
it prints is pretty misleading. If anyone reading this is near a keyboard,
maybe you can file a bug on me for that?
On Mar 5, 2015 1:54 AM, <finnur@chromium.org> wrote:

> This Cl caused the official Linux x64 bot to go red due to the sizes check.
> https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/78934
>
>
> https://codereview.chromium.org/980523002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Nico, 2015-03-05 17:32:54]

Filed https://code.google.com/p/chromium/issues/detail?id=464417

On Thu, Mar 5, 2015 at 6:34 AM, Nico Weber <thakis@chromium.org> wrote:

> It made chrome over 1MB smaller, it's OK if it adds ten kB to nacl_helper
> :-)
>
> That bot really should call eu-strip before printing the size BTW. The
> size it prints is pretty misleading. If anyone reading this is near a
> keyboard, maybe you can file a bug on me for that?
> On Mar 5, 2015 1:54 AM, <finnur@chromium.org> wrote:
>
>> This Cl caused the official Linux x64 bot to go red due to the sizes
>> check.
>> https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/78934
>>
>>
>> https://codereview.chromium.org/980523002/
>>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.