[turbofan] Introduce TypedObjectState common operator.

[turbofan] Introduce TypedObjectState common operator.

This adds a new TypedObjectState operator, which is a version of
ObjectState that carries along MachineTypes for the inputs, so we
can tell the deoptimizer how to interpret the inputs, instead of
having to force everything to Tagged.

Drive-by-fix: Remove the unused id parameter from ObjectState.

R=tebbi@chromium.org
BUG=v8:5609
Committed: https://crrev.com/a5d251defebb69c4a39634fc8de8a6b074c3daf3
Cr-Commit-Position: refs/heads/master@{#40832}

[Benedikt Meurer, 2016-11-08 08:06:32]

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

[commit-bot: I haz the power, 2016-11-08 08:06:41]

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

[Benedikt Meurer, 2016-11-08 08:07:26]

Hey Tobias,

Here's the CL we talked about offline; porting the machinery from
(Typed)StateValues to (Typed)ObjectState as well.
Please take a look.

Thanks,
Benedikt

[commit-bot: I haz the power, 2016-11-08 08:10:51]

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

[commit-bot: I haz the power, 2016-11-08 08:10:52]

Dry run: Try jobs failed on following builders:
  v8_win64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel_ng/builds/17329)

[Benedikt Meurer, 2016-11-08 08:12:12]

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

[commit-bot: I haz the power, 2016-11-08 08:12:21]

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-08 08:50:10]

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

[commit-bot: I haz the power, 2016-11-08 08:50:11]

Dry run: This issue passed the CQ dry run.

[Jarin, 2016-11-08 10:53:07]

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

[Jarin, 2016-11-08 10:53:08]

lgtm

[Tobias Tebbi, 2016-11-08 13:41:40]

lgtm

https://codereview.chromium.org/2488623002/diff/20001/src/compiler/common-ope...
File src/compiler/common-operator.cc (right):

https://codereview.chromium.org/2488623002/diff/20001/src/compiler/common-ope...
src/compiler/common-operator.cc:1027: pointer_slots);                          
// parameter
Why put pointer_slots in this apparently unused parameter?

[Benedikt Meurer, 2016-11-08 13:42:44]

https://codereview.chromium.org/2488623002/diff/20001/src/compiler/common-ope...
File src/compiler/common-operator.cc (right):

https://codereview.chromium.org/2488623002/diff/20001/src/compiler/common-ope...
src/compiler/common-operator.cc:1027: pointer_slots);                          
// parameter
We need to distinguish the operators for hashing and equality. Otherwise the
ObjectState(1) and ObjectState(2) operators are considered equal.

[Benedikt Meurer, 2016-11-08 13:42:52]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-11-08 13:43: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-11-08 13:45:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:26:03]

Description was changed from

==========
[turbofan] Introduce TypedObjectState common operator.

This adds a new TypedObjectState operator, which is a version of
ObjectState that carries along MachineTypes for the inputs, so we
can tell the deoptimizer how to interpret the inputs, instead of
having to force everything to Tagged.

Drive-by-fix: Remove the unused id parameter from ObjectState.

R=tebbi@chromium.org
BUG=v8:5609
==========

to

==========
[turbofan] Introduce TypedObjectState common operator.

This adds a new TypedObjectState operator, which is a version of
ObjectState that carries along MachineTypes for the inputs, so we
can tell the deoptimizer how to interpret the inputs, instead of
having to force everything to Tagged.

Drive-by-fix: Remove the unused id parameter from ObjectState.

R=tebbi@chromium.org
BUG=v8:5609

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

[commit-bot: I haz the power, 2016-11-17 22:26:04]

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