[wasm] Add a flag --wasm-opt to test optimizations in the WASM pipeline.

[wasm] Add a flag --wasm-opt to test optimizations in the WASM pipeline.

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

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

[titzer, 2016-11-24 16:36:31]

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

[commit-bot: I haz the power, 2016-11-24 16:36:38]

Dry run: 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-24 16:53:06]

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

[commit-bot: I haz the power, 2016-11-24 16:53:07]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[bradnelson, 2016-11-24 20:53:48]

lgtm

[ahaas, 2016-11-28 08:34:43]

lgtm with questions

https://codereview.chromium.org/2528033002/diff/1/src/compiler/machine-operat...
File src/compiler/machine-operator-reducer.cc (right):

https://codereview.chromium.org/2528033002/diff/1/src/compiler/machine-operat...
src/compiler/machine-operator-reducer.cc:53: Diamond d(graph(), common(),
Why do you do this change in this CL?

https://codereview.chromium.org/2528033002/diff/1/src/compiler/pipeline.cc
File src/compiler/pipeline.cc (right):

https://codereview.chromium.org/2528033002/diff/1/src/compiler/pipeline.cc#ne...
src/compiler/pipeline.cc:127: javascript_(jsgraph->javascript()),
I think jsgraph->javascript() should always be null. Should we check this with a
DCHECK?

[titzer, 2016-11-28 09:30:22]

https://codereview.chromium.org/2528033002/diff/1/src/compiler/machine-operat...
File src/compiler/machine-operator-reducer.cc (right):

https://codereview.chromium.org/2528033002/diff/1/src/compiler/machine-operat...
src/compiler/machine-operator-reducer.cc:53: Diamond d(graph(), common(),
On 2016/11/28 08:34:43, ahaas wrote:
> Why do you do this change in this CL?

Because otherwise, with selects in the graph, we need to run a further pass on
the graph to do select lowering.

[titzer, 2016-11-28 09:31:54]

https://codereview.chromium.org/2528033002/diff/1/src/compiler/pipeline.cc
File src/compiler/pipeline.cc (right):

https://codereview.chromium.org/2528033002/diff/1/src/compiler/pipeline.cc#ne...
src/compiler/pipeline.cc:127: javascript_(jsgraph->javascript()),
On 2016/11/28 08:34:43, ahaas wrote:
> I think jsgraph->javascript() should always be null. Should we check this with
a
> DCHECK?

I think that would just made the code brittle without checking anything of
interest.

[titzer, 2016-11-28 10:13:45]

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

[commit-bot: I haz the power, 2016-11-28 10:13:53]

Dry run: 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-28 10:42:23]

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

[commit-bot: I haz the power, 2016-11-28 10:42:24]

Dry run: This issue passed the CQ dry run.

[titzer, 2016-11-30 10:04:09]

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

[commit-bot: I haz the power, 2016-11-30 10:04:11]

Dry run: 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-30 10:31:21]

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

[commit-bot: I haz the power, 2016-11-30 10:31:22]

Dry run: This issue passed the CQ dry run.

[titzer, 2016-11-30 10:34:15]

The CQ bit was checked by titzer@chromium.org

[titzer, 2016-11-30 10:34:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
bradnelson@chromium.org, ahaas@chromium.org
Link to the patchset: https://codereview.chromium.org/2528033002/#ps40001
(title: "[wasm] Add a flag --wasm-opt to test optimizations in the WASM
pipeline.")

[commit-bot: I haz the power, 2016-11-30 10:34:18]

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-30 10:35:54]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480502055738360,
"parent_rev": "3bd5e2b1ffc9e930479a3bb92fcad5fc8a7b25e9", "commit_rev":
"c742427199cccee38eecde32afd2f1764150e16a"}

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

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-30 10:36:22]

Description was changed from

==========
[wasm] Add a flag --wasm-opt to test optimizations in the WASM pipeline.

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

to

==========
[wasm] Add a flag --wasm-opt to test optimizations in the WASM pipeline.

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

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

[commit-bot: I haz the power, 2016-11-30 10:36:23]

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