[ic] Don't share LoadGlobalIC slots inside typeof and outside typeof.

[ic] Don't share LoadGlobalIC slots inside typeof and outside typeof.

Because in case of interceptors we will install a slow stub that suits only one case.

BUG=chromium:634467
TBR=verwaest@chromium.org

Committed: https://crrev.com/d634e65fb07fb7c3a79680237c5e90a9fae6645c
Cr-Commit-Position: refs/heads/master@{#38503}

[Igor Sheludko, 2016-08-06 21:09:35]

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

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

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-08-06 21:28:55]

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

[commit-bot: I haz the power, 2016-08-06 21:28:56]

Dry run: This issue passed the CQ dry run.

[Igor Sheludko, 2016-08-07 20:14:25]

ishell@chromium.org changed reviewers:
+ mvstanton@chromium.org

[Igor Sheludko, 2016-08-07 20:14:26]

PTAL

[Igor Sheludko, 2016-08-08 01:46:19]

On 2016/08/07 20:14:26, Igor Sheludko wrote:
> PTAL

Another option would be to merge LoadGlobalIC_SlowInsideTypeof and
LoadGlobalIC_SlowNotInsideTypeof together and let it query LoadGlobalICState if
the property is not found.

[Igor Sheludko, 2016-08-09 15:56:23]

ishell@chromium.org changed reviewers:
+ jkummerow@chromium.org

[Igor Sheludko, 2016-08-09 15:56:24]

Jakob, PTAL

[Jakob Kummerow, 2016-08-09 15:58:47]

lgtm

[Igor Sheludko, 2016-08-09 16:10:07]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-08-09 16:10:17]

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-09 16:12:47]

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

[commit-bot: I haz the power, 2016-08-09 16:12:48]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/21429)

[Igor Sheludko, 2016-08-09 16:26:40]

Description was changed from

==========
[ic] Don't share LoadGlobalIC slots inside typeof and outside typeof.

Because in case of interceptors we will install a slow stub that suits only one
case.

BUG=chromium:634467
==========

to

==========
[ic] Don't share LoadGlobalIC slots inside typeof and outside typeof.

Because in case of interceptors we will install a slow stub that suits only one
case.

BUG=chromium:634467
TBR=verwaest@chromium.org
==========

[Igor Sheludko, 2016-08-09 16:26:46]

The CQ bit was checked by ishell@chromium.org

[commit-bot: I haz the power, 2016-08-09 16:26:53]

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-09 16:30:33]

Description was changed from

==========
[ic] Don't share LoadGlobalIC slots inside typeof and outside typeof.

Because in case of interceptors we will install a slow stub that suits only one
case.

BUG=chromium:634467
TBR=verwaest@chromium.org
==========

to

==========
[ic] Don't share LoadGlobalIC slots inside typeof and outside typeof.

Because in case of interceptors we will install a slow stub that suits only one
case.

BUG=chromium:634467
TBR=verwaest@chromium.org
==========

[commit-bot: I haz the power, 2016-08-09 16:30:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-09 16:30:53]

Description was changed from

==========
[ic] Don't share LoadGlobalIC slots inside typeof and outside typeof.

Because in case of interceptors we will install a slow stub that suits only one
case.

BUG=chromium:634467
TBR=verwaest@chromium.org
==========

to

==========
[ic] Don't share LoadGlobalIC slots inside typeof and outside typeof.

Because in case of interceptors we will install a slow stub that suits only one
case.

BUG=chromium:634467
TBR=verwaest@chromium.org

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

[commit-bot: I haz the power, 2016-08-09 16:30:54]

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