Compile asm->wasm conversion at parse time.

Compile asm->wasm conversion at parse time.

Move compiltion of asm.js code to parse time.

BUG= https://bugs.chromium.org/p/v8/issues/detail?id=4203
TEST=asm-wasm
R=mtrofin@chromium.org
LOG=N

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

[bradn, 2016-07-12 01:01:22]

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

[bradn, 2016-07-12 01:01:22]

Hi Mircea.
This has failures for two tests.
mjsunit/asm/* --extra-flags=--validate-asm
There's likely a race involved, as the dcheck that's failing, doesn't seem to
have that value any longer by the time you catch it in the debugger

[Mircea Trofin, 2016-07-12 22:07:34]

lgtm

https://codereview.chromium.org/2141813002/diff/1/src/asmjs/asm-js.cc
File src/asmjs/asm-js.cc (right):

https://codereview.chromium.org/2141813002/diff/1/src/asmjs/asm-js.cc#newcode79
src/asmjs/asm-js.cc:79: CompileModule(info->isolate(), module->begin(),
module->end(), &thrower,
To echo our chat - this change moves compiling at parse time instead
of runtime. Compile time is lengthy and may impact user experience. Would this
change surprise or block a developer in terms of their ability to mitigate
compile time? Do we need to pair it with guidance?

[bradn, 2016-07-12 22:34:41]

It will have impact, but only for now with this flag on.
We should definitely communicate on the change as this gets rolled out. Which
will probably happen incrementally.

[bradn, 2016-07-12 22:34:49]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2016-07-12 22:35:24]

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-07-12 23:10:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-12 23:12:25]

Description was changed from

==========
Compile asm->wasm conversion at parse time.

Move compiltion of asm.js code to parse time.

BUG= https://bugs.chromium.org/p/v8/issues/detail?id=4203
TEST=asm-wasm
R=mtrofin@chromium.org
LOG=N
==========

to

==========
Compile asm->wasm conversion at parse time.

Move compiltion of asm.js code to parse time.

BUG= https://bugs.chromium.org/p/v8/issues/detail?id=4203
TEST=asm-wasm
R=mtrofin@chromium.org
LOG=N

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

[commit-bot: I haz the power, 2016-07-12 23:12:26]

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