[asm] Don't try to always-opt code which has AsmWasm data.

[asm] Don't try to always-opt code which has AsmWasm data.

If code has AsmWasm data we shouldn't try to optimize the JS as well, since
it will instead be optimized using the WASM data.

BUG=

Committed: https://crrev.com/f666bff8d6c44dbb063560610341233062731f07
Cr-Commit-Position: refs/heads/master@{#41293}

[rmcilroy, 2016-11-25 15:43:50]

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

[commit-bot: I haz the power, 2016-11-25 15:43:55]

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

[rmcilroy, 2016-11-25 15:46:43]

rmcilroy@chromium.org changed reviewers:
+ bmeurer@chromium.org

[rmcilroy, 2016-11-25 15:46:44]

Benedkit, it turns out that this change is also necessary to unblock your CL at
https://codereview.chromium.org/2525243002/. The issue we haven't compiled
bytecode for the function (because it was valid asmjs and so got AsmWasmData
instead). Then, we call GetOptimizedCode in CompileLazy (due to --always-opt),
but since we haven't compiled bytecode, for a function which should go via the
BytecodeGraphBuilder, we end up with the DCHECK you were seeing in the above CL.

PTAL, thanks.

[commit-bot: I haz the power, 2016-11-25 16:13:06]

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

[commit-bot: I haz the power, 2016-11-25 16:13:07]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-11-25 18:24:27]

Nice catch. LGTM.

[rmcilroy, 2016-11-25 20:54:59]

+bradnelson@ as FYI.

[rmcilroy, 2016-11-25 20:55:04]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2016-11-25 20:55:07]

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-11-26 04:46:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480107304214160, "parent_rev":
"b98c7c4d577a33750220dfc89d1029ed2bb6fb78", "commit_rev":
"963c1c542241ecef11e4e865b3b7aa07d9e30492"}

[commit-bot: I haz the power, 2016-11-26 04:46:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-26 04:47:07]

Description was changed from

==========
[asm] Don't try to always-opt code which has AsmWasm data.

If code has AsmWasm data we shouldn't try to optimize the JS as well, since
it will instead be optimized using the WASM data.

BUG=
==========

to

==========
[asm] Don't try to always-opt code which has AsmWasm data.

If code has AsmWasm data we shouldn't try to optimize the JS as well, since
it will instead be optimized using the WASM data.

BUG=

Committed: https://crrev.com/f666bff8d6c44dbb063560610341233062731f07
Cr-Commit-Position: refs/heads/master@{#41293}
==========

[commit-bot: I haz the power, 2016-11-26 04:47:08]

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