GN: ccache+clang needs -Qunused-arguments

GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

(CCACHE_CPP2 may turn out to be required anyway
if we start seeing warnings about tautological compares.)

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002

TBR=jcgregorio@google.com
NOTRY=true

Committed: https://skia.googlesource.com/skia/+/830c913625ac62b16b09cc0f1dbd8f177ec493d9

[mtklein_C, 2016-07-29 12:47:48]

Description was changed from

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

BUG=skia:
==========

to

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002
==========

[mtklein, 2016-07-29 12:48:12]

Description was changed from

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002
==========

to

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

CCACHE_CPP2 may turn out to be required anyway.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002
==========

[mtklein_C, 2016-07-29 12:53:06]

Description was changed from

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

CCACHE_CPP2 may turn out to be required anyway.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002
==========

to

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

(CCACHE_CPP2 may turn out to be required anyway
if we start seeing warnings about tautological compares.)

This gets these bots past compilation errors and to link errors.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002
==========

[mtklein_C, 2016-07-29 12:58:27]

Description was changed from

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

(CCACHE_CPP2 may turn out to be required anyway
if we start seeing warnings about tautological compares.)

This gets these bots past compilation errors and to link errors.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002
==========

to

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

(CCACHE_CPP2 may turn out to be required anyway
if we start seeing warnings about tautological compares.)

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002
==========

[mtklein_C, 2016-07-29 13:02:50]

Description was changed from

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

(CCACHE_CPP2 may turn out to be required anyway
if we start seeing warnings about tautological compares.)

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002
==========

to

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

(CCACHE_CPP2 may turn out to be required anyway
if we start seeing warnings about tautological compares.)

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002

TBR=jcgregorio@google.com
NOTRY=true
==========

[mtklein_C, 2016-07-29 13:02:53]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-07-29 13:02:58]

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

[mtklein_C, 2016-07-29 13:02:59]

mtklein@chromium.org changed reviewers:
+ jcgregorio@google.com

[commit-bot: I haz the power, 2016-07-29 13:03:54]

Description was changed from

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

(CCACHE_CPP2 may turn out to be required anyway
if we start seeing warnings about tautological compares.)

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002

TBR=jcgregorio@google.com
NOTRY=true
==========

to

==========
GN: ccache+clang needs -Qunused-arguments

An alternative is to define CCACHE_CPP2, but
I thought I'd try this more visible way first.

(CCACHE_CPP2 may turn out to be required anyway
if we start seeing warnings about tautological compares.)

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193203002

TBR=jcgregorio@google.com
NOTRY=true

Committed:
https://skia.googlesource.com/skia/+/830c913625ac62b16b09cc0f1dbd8f177ec493d9
==========

[commit-bot: I haz the power, 2016-07-29 13:03:55]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/830c913625ac62b16b09cc0f1dbd8f177ec493d9

[jcgregorio, 2016-07-29 13:57:54]

lgtm