replace COMPILE_ASSERT with static_assert in gpu/

replace COMPILE_ASSERT with static_assert in gpu/

BUG=442514
Committed: https://crrev.com/7f032091dad424a76cb267772c27b9b165ed157f
Cr-Commit-Position: refs/heads/master@{#309311}

[Mostyn Bramley-Moore, 2014-12-19 22:46:24]

mostynb@opera.com changed reviewers:
+ sievers@chromium.org

[Mostyn Bramley-Moore, 2014-12-19 22:46:25]

@sievers: PTAL.

[no sievers, 2014-12-19 23:28:25]

lgtm

The strings don't seem to add any value though, since they are just restating
the condition, but do make everything look more cluttered. Maybe
static_cast(cond) is good enough actually.

[no sievers, 2014-12-19 23:29:09]

On 2014/12/19 23:28:25, sievers wrote:
> lgtm
> 
> The strings don't seem to add any value though, since they are just restating
> the condition, but do make everything look more cluttered. Maybe
> static_cast(cond) is good enough actually.

my bad, that is C++17 only :)

[Mostyn Bramley-Moore, 2014-12-19 23:38:48]

On 2014/12/19 23:28:25, sievers wrote:
> lgtm
> 
> The strings don't seem to add any value though, since they are just restating
> the condition, but do make everything look more cluttered. Maybe
> static_cast(cond) is good enough actually.

The strings should give a slightly more readable error message than the mix of
camel case/snake case that we currently have (but they do appear redundant if
you're looking at the source code).

[Mostyn Bramley-Moore, 2014-12-19 23:38:59]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2014-12-19 23:39:21]

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

[commit-bot: I haz the power, 2014-12-20 00:37:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-12-20 00:38:11]

Patchset 1 (id:??) landed as
https://crrev.com/7f032091dad424a76cb267772c27b9b165ed157f
Cr-Commit-Position: refs/heads/master@{#309311}