[turbofan] Properly assign types to Array/String iterators.

[turbofan] Properly assign types to Array/String iterators.

When inlining allocations for Array and String iterators, make sure the
allocations have proper types on it (rather than just Type::Any). This
allows us to escape analyze the iterators in a couple of more cases,
i.e. in this very simple case:

  function f(a) {
    const it = a[Symbol.iterator]();
    return it.next().value;
  }

R=jarin@chromium.org
BUG=v8:4586, v8:5448
Review-Url: https://codereview.chromium.org/2640743003
Cr-Commit-Position: refs/heads/master@{#42449}
Committed: https://chromium.googlesource.com/v8/v8/+/977038516bbdd0427eacf8be4496e63b451f3ffe

[Benedikt Meurer, 2017-01-18 10:05:15]

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

[commit-bot: I haz the power, 2017-01-18 10:05:26]

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

[Benedikt Meurer, 2017-01-18 10:14:10]

Hey Jaro,

Here's the fix for the escaping iterator in the trivial case.
Please take a look.

Thanks,
Benedikt

[Benedikt Meurer, 2017-01-18 10:14:10]

Hey Jaro,

Here's the fix for the escaping iterator in the trivial case.
Please take a look.

Thanks,
Benedikt

[commit-bot: I haz the power, 2017-01-18 10:19:55]

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

[commit-bot: I haz the power, 2017-01-18 10:19:56]

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...)

[Benedikt Meurer, 2017-01-18 10:32:07]

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

[commit-bot: I haz the power, 2017-01-18 10:32:25]

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

[Jarin, 2017-01-18 10:48:02]

lgtm

[commit-bot: I haz the power, 2017-01-18 10:57:38]

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

[commit-bot: I haz the power, 2017-01-18 10:57:39]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2017-01-18 11:22:18]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2017-01-18 11:22:24]

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-01-18 11:24:01]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484738538562780,
"parent_rev": "a45fa2d856908852f0264391e6b05fbe6611b2ac", "commit_rev":
"977038516bbdd0427eacf8be4496e63b451f3ffe"}

[commit-bot: I haz the power, 2017-01-18 11:24:04]

Description was changed from

==========
[turbofan] Properly assign types to Array/String iterators.

When inlining allocations for Array and String iterators, make sure the
allocations have proper types on it (rather than just Type::Any). This
allows us to escape analyze the iterators in a couple of more cases,
i.e. in this very simple case:

  function f(a) {
    const it = a[Symbol.iterator]();
    return it.next().value;
  }

R=jarin@chromium.org
BUG=v8:4586,v8:5448
==========

to

==========
[turbofan] Properly assign types to Array/String iterators.

When inlining allocations for Array and String iterators, make sure the
allocations have proper types on it (rather than just Type::Any). This
allows us to escape analyze the iterators in a couple of more cases,
i.e. in this very simple case:

  function f(a) {
    const it = a[Symbol.iterator]();
    return it.next().value;
  }

R=jarin@chromium.org
BUG=v8:4586,v8:5448

Review-Url: https://codereview.chromium.org/2640743003
Cr-Commit-Position: refs/heads/master@{#42449}
Committed:
https://chromium.googlesource.com/v8/v8/+/977038516bbdd0427eacf8be4496e63b451...
==========

[commit-bot: I haz the power, 2017-01-18 11:24:05]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/977038516bbdd0427eacf8be4496e63b451...