[turbofan] Fix merging of empty and non-empty state in load elimination.

[turbofan] Fix merging of empty and non-empty state in load elimination.

Unfortunately, I was unable to produce a repro without asm.js. In normal
JavaScript, the bounds check renaming saves us.

I have not done anything about the index variable aliasing and handling
of differently sized elements yet!

BUG=chromium:639210, v8:5266
Committed: https://crrev.com/dc330f2bd35656ecc133f7f329edee302a809935
Cr-Commit-Position: refs/heads/master@{#38874}

[Jarin, 2016-08-24 15:03:22]

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

[commit-bot: I haz the power, 2016-08-24 15:03:33]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Jarin, 2016-08-24 15:09:48]

jarin@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Jarin, 2016-08-24 15:09:49]

Could you take a look, please?

[commit-bot: I haz the power, 2016-08-24 15:23:23]

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

[commit-bot: I haz the power, 2016-08-24 15:23:24]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-08-24 17:11:37]

Description was changed from

==========
[turbofan] Fix merging of empty and non-empty state in load elimination.

Unfortunately, I was unable to produce a repro without asm.js. In normal
JavaScript, the bounds check renaming saves us.

I have not done anything about the index variable aliasing and handling
of differently sized elements yet!

BUG=chromium:639210
==========

to

==========
[turbofan] Fix merging of empty and non-empty state in load elimination.

Unfortunately, I was unable to produce a repro without asm.js. In normal
JavaScript, the bounds check renaming saves us.

I have not done anything about the index variable aliasing and handling
of differently sized elements yet!

BUG=chromium:639210, v8:5266
==========

[Benedikt Meurer, 2016-08-24 17:11:49]

Awesome, thanks. LGTM.

[Benedikt Meurer, 2016-08-24 17:11:54]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-08-24 17:12: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, 2016-08-24 17:14:01]

Description was changed from

==========
[turbofan] Fix merging of empty and non-empty state in load elimination.

Unfortunately, I was unable to produce a repro without asm.js. In normal
JavaScript, the bounds check renaming saves us.

I have not done anything about the index variable aliasing and handling
of differently sized elements yet!

BUG=chromium:639210, v8:5266
==========

to

==========
[turbofan] Fix merging of empty and non-empty state in load elimination.

Unfortunately, I was unable to produce a repro without asm.js. In normal
JavaScript, the bounds check renaming saves us.

I have not done anything about the index variable aliasing and handling
of differently sized elements yet!

BUG=chromium:639210, v8:5266
==========

[commit-bot: I haz the power, 2016-08-24 17:14:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-24 17:14:32]

Description was changed from

==========
[turbofan] Fix merging of empty and non-empty state in load elimination.

Unfortunately, I was unable to produce a repro without asm.js. In normal
JavaScript, the bounds check renaming saves us.

I have not done anything about the index variable aliasing and handling
of differently sized elements yet!

BUG=chromium:639210, v8:5266
==========

to

==========
[turbofan] Fix merging of empty and non-empty state in load elimination.

Unfortunately, I was unable to produce a repro without asm.js. In normal
JavaScript, the bounds check renaming saves us.

I have not done anything about the index variable aliasing and handling
of differently sized elements yet!

BUG=chromium:639210, v8:5266

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

[commit-bot: I haz the power, 2016-08-24 17:14:33]

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