[turbofan] Allow stores bigger than tagged size in store-store elimination.

[turbofan] Allow stores bigger than tagged size in store-store elimination.

BUG=

Committed: https://crrev.com/2652812b642f39cdfbe91ca9e0ddfbebdb9d44f8
Cr-Commit-Position: refs/heads/master@{#37374}

[bgeron, 2016-06-28 16:54:24]

bgeron@google.com changed reviewers:
+ jarin@chromium.org

[bgeron, 2016-06-28 16:54:25]

PTAL.

[bgeron, 2016-06-28 16:56:20]

See the dependent patchset for try results.

[bgeron, 2016-06-28 16:57:02]

Tests succeed for Optdebug on x64 and ia32 on my machine. (Contrary to
https://codereview.chromium.org/2099683002/#ps1)

[Jarin, 2016-06-28 21:30:55]

lgtm. Thanks!

https://codereview.chromium.org/2107833002/diff/1/src/compiler/store-store-el...
File src/compiler/store-store-elimination.cc (right):

https://codereview.chromium.org/2107833002/diff/1/src/compiler/store-store-el...
src/compiler/store-store-elimination.cc:210: bool LeqTagged(FieldAccess access)
{
How about AtMostTagged and AtLeastTagged?

[bgeron, 2016-06-29 08:31:00]

Landing now.

https://codereview.chromium.org/2107833002/diff/1/src/compiler/store-store-el...
File src/compiler/store-store-elimination.cc (right):

https://codereview.chromium.org/2107833002/diff/1/src/compiler/store-store-el...
src/compiler/store-store-elimination.cc:210: bool LeqTagged(FieldAccess access)
{
On 2016/06/28 21:30:55, Jarin wrote:
> How about AtMostTagged and AtLeastTagged?

Good idea. Changed.

[bgeron, 2016-06-29 08:31:03]

The CQ bit was checked by bgeron@google.com

[bgeron, 2016-06-29 08:31:03]

The patchset sent to the CQ was uploaded after l-g-t-m from jarin@chromium.org
Link to the patchset: https://codereview.chromium.org/2107833002/#ps20001
(title: "Rename functions as suggested")

[commit-bot: I haz the power, 2016-06-29 08:31: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-06-29 09:26:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-29 09:29:45]

Description was changed from

==========
[turbofan] Allow stores bigger than tagged size in store-store elimination.

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

to

==========
[turbofan] Allow stores bigger than tagged size in store-store elimination.

BUG=

Committed: https://crrev.com/2652812b642f39cdfbe91ca9e0ddfbebdb9d44f8
Cr-Commit-Position: refs/heads/master@{#37374}
==========

[commit-bot: I haz the power, 2016-06-29 09:29:46]

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