Adding flag for simd.js + asm->wasm.

Adding flag for simd.js + asm->wasm.

Make it possible to switch on simd.js support when combined with asm.js
in the asm->wasm path.

BUG= https://code.google.com/p/v8/issues/detail?id=4203
TEST=test-asm-validator
R=gdeepti@chromium.org,titzer@chromium.org
LOG=N

Committed: https://crrev.com/4529cb7b6f00b3513f44c9b1dee0b8fd28bb38ab
Cr-Commit-Position: refs/heads/master@{#33644}

[bradn, 2016-01-29 20:20:26]

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

[gdeepti1, 2016-01-30 01:09:12]

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

[gdeepti1, 2016-01-30 01:09:12]

lgtm

[bradn, 2016-01-30 01:12:03]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2016-01-30 01:12:10]

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

[commit-bot: I haz the power, 2016-01-30 01:15:10]

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

[commit-bot: I haz the power, 2016-01-30 01:15:11]

Try jobs failed on following builders:
  v8_presubmit on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/10332)

[bradn, 2016-01-30 01:48:58]

Need OWNERs review.

[titzer, 2016-02-01 08:04:36]

https://codereview.chromium.org/1643333002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):

https://codereview.chromium.org/1643333002/diff/1/src/flag-definitions.h#newc...
src/flag-definitions.h:487: DEFINE_BOOL(asm_thru_wasm_simd, false, "enable
SIMD.js for asm->wasm")
Flag has a weird name. Suggest "verb_feature" or just "feature" somehow...

Hmm, "enable_simd_asmjs" ?

[bradn, 2016-02-01 17:25:18]

ptal

https://codereview.chromium.org/1643333002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):

https://codereview.chromium.org/1643333002/diff/1/src/flag-definitions.h#newc...
src/flag-definitions.h:487: DEFINE_BOOL(asm_thru_wasm_simd, false, "enable
SIMD.js for asm->wasm")
On 2016/02/01 08:04:36, titzer wrote:
> Flag has a weird name. Suggest "verb_feature" or just "feature" somehow...
> 
> Hmm, "enable_simd_asmjs" ?

Done.

[titzer, 2016-02-01 17:53:05]

lgtm

[bradn, 2016-02-01 17:53:57]

The CQ bit was checked by bradnelson@google.com

[bradn, 2016-02-01 17:53:57]

The patchset sent to the CQ was uploaded after l-g-t-m from gdeepti@google.com
Link to the patchset: https://codereview.chromium.org/1643333002/#ps20001
(title: "fix")

[commit-bot: I haz the power, 2016-02-01 17:54:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1643333002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1643333002/20001

[commit-bot: I haz the power, 2016-02-01 18:25:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-01 18:25:48]

Description was changed from

==========
Adding flag for simd.js + asm->wasm.

Make it possible to switch on simd.js support when combined with asm.js
in the asm->wasm path.

BUG= https://code.google.com/p/v8/issues/detail?id=4203
TEST=test-asm-validator
R=gdeepti@chromium.org,titzer@chromium.org
LOG=N
==========

to

==========
Adding flag for simd.js + asm->wasm.

Make it possible to switch on simd.js support when combined with asm.js
in the asm->wasm path.

BUG= https://code.google.com/p/v8/issues/detail?id=4203
TEST=test-asm-validator
R=gdeepti@chromium.org,titzer@chromium.org
LOG=N

Committed: https://crrev.com/4529cb7b6f00b3513f44c9b1dee0b8fd28bb38ab
Cr-Commit-Position: refs/heads/master@{#33644}
==========

[commit-bot: I haz the power, 2016-02-01 18:25:48]

Patchset 2 (id:??) landed as
https://crrev.com/4529cb7b6f00b3513f44c9b1dee0b8fd28bb38ab
Cr-Commit-Position: refs/heads/master@{#33644}