[ia32] Optimize index calculation for certain checked load/stores.

[ia32] Optimize index calculation for certain checked load/stores.

It reduces the add operation for certain checked load/stores by pre-do
the add in compiling phase.


BUG=

Review-Url: https://codereview.chromium.org/2593483002
Cr-Commit-Position: refs/heads/master@{#43128}
Committed: https://chromium.googlesource.com/v8/v8/+/d401543f84a0b9fa210a8d3c4330bbfe0f8788f4

[shiyu.zhang, 2016-12-20 09:28:10]

Patchset #5 (id:80001) has been deleted

[shiyu.zhang, 2016-12-20 09:28:29]

Patchset #4 (id:60001) has been deleted

[shiyu.zhang, 2016-12-21 06:40:34]

Description was changed from

==========
[ia32] Optimize index calculation for certain checked load/stores.

It reduces the add operation for certain checked load/stores by pre-do
the add in compiling phase.


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

to

==========
[ia32] Optimize index calculation for certain checked load/stores.

It reduces the add operation for certain checked load/stores by pre-do
the add in compiling phase.


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

[shiyu.zhang, 2016-12-21 06:40:34]

shiyu.zhang@intel.com changed reviewers:
+ bmeurer@chromium.org, bradnelson@chromium.org, jarin@chromium.org

[shiyu.zhang, 2016-12-21 06:45:48]

This patch fixed the regressions in
https://bugs.chromium.org/p/chromium/issues/detail?id=673861. What's more, it
applied the optimization for certain checked load/stores of
https://codereview.chromium.org/2563483005/ into asm-wasm pipeline too. Thus,
~7% improvement is observed for embenchen-skinning.js, jetstream-dry.c and
jetstream-gcc-loops.cpp when enabling asw-wasm (--validate-asm) on ia32 HSW.
Appreciate your comments, thanks! PTAL

[shiyu.zhang, 2016-12-23 03:39:59]

@Benedikt Meurer, PTAL

[danno, 2017-01-19 15:47:49]

danno@chromium.org changed reviewers:
+ ahaas@chromium.org, danno@chromium.org, titzer@google.com

[titzer, 2017-01-19 15:49:42]

titzer@chromium.org changed reviewers:
+ titzer@chromium.org
- titzer@google.com

[bradnelson, 2017-02-12 08:16:34]

The CQ bit was checked by bradnelson@chromium.org

[bradnelson, 2017-02-12 08:16:35]

lgtm

[commit-bot: I haz the power, 2017-02-12 08:16:42]

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, 2017-02-12 08:19:31]

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

[commit-bot: I haz the power, 2017-02-12 08:19:32]

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

[bradn, 2017-02-12 08:34:46]

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

[bradn, 2017-02-12 08:34:46]

bmeurer, will need your OWNERS blessing too.

[Benedikt Meurer, 2017-02-12 10:52:37]

On 2017/02/12 08:34:46, bradn wrote:
> bmeurer, will need your OWNERS blessing too.

LGTM (rubber stamped)

[bradn, 2017-02-12 16:20:49]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2017-02-12 16:21:01]

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, 2017-02-12 16:24:14]

CQ is committing da patch.

Bot data: {"patchset_id": 160001, "attempt_start_ts": 1486916449902900,
"parent_rev": "340c64af4b7c1b7794e267aba71d6aa43ce9a708", "commit_rev":
"d401543f84a0b9fa210a8d3c4330bbfe0f8788f4"}

[commit-bot: I haz the power, 2017-02-12 16:24:23]

Description was changed from

==========
[ia32] Optimize index calculation for certain checked load/stores.

It reduces the add operation for certain checked load/stores by pre-do
the add in compiling phase.


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

to

==========
[ia32] Optimize index calculation for certain checked load/stores.

It reduces the add operation for certain checked load/stores by pre-do
the add in compiling phase.


BUG=

Review-Url: https://codereview.chromium.org/2593483002
Cr-Commit-Position: refs/heads/master@{#43128}
Committed:
https://chromium.googlesource.com/v8/v8/+/d401543f84a0b9fa210a8d3c4330bbfe0f8...
==========

[commit-bot: I haz the power, 2017-02-12 16:24:26]

Committed patchset #7 (id:160001) as
https://chromium.googlesource.com/v8/v8/+/d401543f84a0b9fa210a8d3c4330bbfe0f8...