[turbofan] Remove over-restrictive DCHECKs.

[turbofan] Remove over-restrictive DCHECKs.

The KeyedStoreMode that we get out of the FeedbackNexus doesn't
necessarily need to apply when we have "static knowledge" about
the receiver, i.e. when the receiver is a known JSTypedArray, but
the KEYED_STORE_IC has seen only JSArray instances so far. The
DCHECK was too restrictive in this case, since we can just ignore
the KEYED_STORE_IC mode (like we ignore the maps).

BUG=chromium:685050
R=ishell@chromium.org

Review-Url: https://codereview.chromium.org/2668643002
Cr-Commit-Position: refs/heads/master@{#42810}
Committed: https://chromium.googlesource.com/v8/v8/+/64eae6eff294670c4503d9a95fbe164e34edcdf1

[Benedikt Meurer, 2017-01-31 07:38:23]

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

[commit-bot: I haz the power, 2017-01-31 07:38:27]

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-31 07:38:51]

Hey Igor,

Easy one.
Please take a look.

Thanks,
Benedikt

[commit-bot: I haz the power, 2017-01-31 08:05:51]

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

[commit-bot: I haz the power, 2017-01-31 08:05:52]

Dry run: This issue passed the CQ dry run.

[Igor Sheludko, 2017-01-31 08:31:54]

lgtm

[Benedikt Meurer, 2017-01-31 08:58:51]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2017-01-31 08:59:00]

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-31 09:00:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485853131122820, "parent_rev":
"68ae57ce1e220196b73b7c9c031756421291a24f", "commit_rev":
"64eae6eff294670c4503d9a95fbe164e34edcdf1"}

[commit-bot: I haz the power, 2017-01-31 09:00:59]

Description was changed from

==========
[turbofan] Remove over-restrictive DCHECKs.

The KeyedStoreMode that we get out of the FeedbackNexus doesn't
necessarily need to apply when we have "static knowledge" about
the receiver, i.e. when the receiver is a known JSTypedArray, but
the KEYED_STORE_IC has seen only JSArray instances so far. The
DCHECK was too restrictive in this case, since we can just ignore
the KEYED_STORE_IC mode (like we ignore the maps).

BUG=chromium:685050
R=ishell@chromium.org
==========

to

==========
[turbofan] Remove over-restrictive DCHECKs.

The KeyedStoreMode that we get out of the FeedbackNexus doesn't
necessarily need to apply when we have "static knowledge" about
the receiver, i.e. when the receiver is a known JSTypedArray, but
the KEYED_STORE_IC has seen only JSArray instances so far. The
DCHECK was too restrictive in this case, since we can just ignore
the KEYED_STORE_IC mode (like we ignore the maps).

BUG=chromium:685050
R=ishell@chromium.org

Review-Url: https://codereview.chromium.org/2668643002
Cr-Commit-Position: refs/heads/master@{#42810}
Committed:
https://chromium.googlesource.com/v8/v8/+/64eae6eff294670c4503d9a95fbe164e34e...
==========

[commit-bot: I haz the power, 2017-01-31 09:01:00]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/64eae6eff294670c4503d9a95fbe164e34e...