[wasm] Only do SIMD lowering if SIMD is present in the graph.

[wasm] Only do SIMD lowering if SIMD is present in the graph.

R=bradnelson@chromium.org, ahaas@chromium.org
BUG=

Committed: https://crrev.com/768acf683bd5059f467c72700a9c3d36b863faa8
Cr-Commit-Position: refs/heads/master@{#41621}

[titzer, 2016-12-09 14:22:57]

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

[commit-bot: I haz the power, 2016-12-09 14:22:59]

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

[titzer, 2016-12-09 14:29:06]

PTAL

[commit-bot: I haz the power, 2016-12-09 14:48:51]

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

[commit-bot: I haz the power, 2016-12-09 14:48:52]

Dry run: This issue passed the CQ dry run.

[bradn, 2016-12-09 14:49:04]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com

[bradn, 2016-12-09 14:49:05]

lgtm

https://codereview.chromium.org/2568493002/diff/1/src/compiler/wasm-compiler.cc
File src/compiler/wasm-compiler.cc (right):

https://codereview.chromium.org/2568493002/diff/1/src/compiler/wasm-compiler....
src/compiler/wasm-compiler.cc:3450: if (builder.has_simd() &&
!CpuFeatures::SupportsSimd128()) {
Oops, sorry thought I'd asked gdeepti / aseemgarg to add this sort of thing, or
at least gate these off based on the SIMD flag!
Thanks!

https://codereview.chromium.org/2568493002/diff/1/src/compiler/wasm-compiler.h
File src/compiler/wasm-compiler.h (left):

https://codereview.chromium.org/2568493002/diff/1/src/compiler/wasm-compiler....
src/compiler/wasm-compiler.h:243: wasm::FunctionSig* function_signature_;
https://google.github.io/styleguide/cppguide.html#General_Naming_Rules

[ahaas, 2016-12-09 14:51:20]

ahaas@chromium.org changed reviewers:
- bradnelson@google.com

[ahaas, 2016-12-09 14:51:20]

lgtm

[titzer, 2016-12-09 14:54:59]

The CQ bit was checked by titzer@chromium.org

[commit-bot: I haz the power, 2016-12-09 14:55:04]

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, 2016-12-09 14:56:57]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481295299430690, "parent_rev":
"ae555d97463b50202f3d6ed4f764b5605efad61d", "commit_rev":
"a2a19afa5e3a063a6978a86d5abdd2e7dfafc659"}

[commit-bot: I haz the power, 2016-12-09 14:57:03]

Description was changed from

==========
[wasm] Only do SIMD lowering if SIMD is present in the graph.

R=bradnelson@chromium.org, ahaas@chromium.org
BUG=
==========

to

==========
[wasm] Only do SIMD lowering if SIMD is present in the graph.

R=bradnelson@chromium.org, ahaas@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2568493002
==========

[commit-bot: I haz the power, 2016-12-09 14:57:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-09 14:57:18]

Description was changed from

==========
[wasm] Only do SIMD lowering if SIMD is present in the graph.

R=bradnelson@chromium.org, ahaas@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2568493002
==========

to

==========
[wasm] Only do SIMD lowering if SIMD is present in the graph.

R=bradnelson@chromium.org, ahaas@chromium.org
BUG=

Committed: https://crrev.com/768acf683bd5059f467c72700a9c3d36b863faa8
Cr-Commit-Position: refs/heads/master@{#41621}
==========

[commit-bot: I haz the power, 2016-12-09 14:57:19]

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