MIPS: Fix for instanceof-proxies failure.

MIPS: Fix for instanceof-proxies failure.

Invalidate the instanceof cache in proxy-case of InstanceOfStub
after the changes in https://codereview.chromium.org/1492243003.

This didn't have a test case until https://codereview.chromium.org/1518773003

TEST=mjsunit/es6/instanceof-proxies.js
BUG=

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

[paul.l..., 2015-12-12 02:26:48]

paul.lind@imgtec.com changed reviewers:
+ akos.palfi@imgtec.com, balazs.kilvady@imgtec.com, bmeurer@chromium.org,
mvstanton@chromium.org

[paul.l..., 2015-12-12 02:26:49]

PTAL.

[akos.palfi.imgtec, 2015-12-12 02:37:28]

LGTM

[paul.l..., 2015-12-12 02:40:17]

The CQ bit was checked by paul.lind@imgtec.com

[dusan.milosavljevic, 2015-12-12 02:40:25]

dusan.m.milosavljevic@gmail.com changed reviewers:
+ dusan.m.milosavljevic@gmail.com

[dusan.milosavljevic, 2015-12-12 02:40:25]

https://codereview.chromium.org/1519303002/diff/1/src/mips/code-stubs-mips.cc
File src/mips/code-stubs-mips.cc (right):

https://codereview.chromium.org/1519303002/diff/1/src/mips/code-stubs-mips.cc...
src/mips/code-stubs-mips.cc:1533: __ li(scratch, Operand(Smi::FromInt(0)));
Can we use zero reg here?

[commit-bot: I haz the power, 2015-12-12 02:40:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1519303002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1519303002/1

[paul.l..., 2015-12-12 02:55:27]

https://codereview.chromium.org/1519303002/diff/1/src/mips/code-stubs-mips.cc
File src/mips/code-stubs-mips.cc (right):

https://codereview.chromium.org/1519303002/diff/1/src/mips/code-stubs-mips.cc...
src/mips/code-stubs-mips.cc:1533: __ li(scratch, Operand(Smi::FromInt(0)));
On 2015/12/12 02:40:25, dusan.milosavljevic wrote:
> Can we use zero reg here?

Definitely we could use move from zero reg. But li() will be encoded as
addiu(scratch, zero_reg, 0), which is single instruction, and unlikely to take
any longer to execute (no memory refs). (With the mov() version, I'd feel
obligated to add an Assert that Smi::FromInt(0) is really zero:) Anyway, I don't
see a payoff for the mov(), do you?

[paul.l..., 2015-12-12 02:56:08]

The CQ bit was unchecked by paul.lind@imgtec.com

[paul.l..., 2015-12-12 02:57:12]

https://codereview.chromium.org/1519303002/diff/1/src/mips/code-stubs-mips.cc
File src/mips/code-stubs-mips.cc (right):

https://codereview.chromium.org/1519303002/diff/1/src/mips/code-stubs-mips.cc...
src/mips/code-stubs-mips.cc:1533: __ li(scratch, Operand(Smi::FromInt(0)));
On 2015/12/12 02:55:27, paul.l... wrote:
> On 2015/12/12 02:40:25, dusan.milosavljevic wrote:
> > Can we use zero reg here?
Sorry, you are right, I missed it. I think I stopped the commit, I'll update the
CL.

[paul.l..., 2015-12-12 03:12:33]

The CQ bit was checked by paul.lind@imgtec.com

[paul.l..., 2015-12-12 03:12:34]

The patchset sent to the CQ was uploaded after l-g-t-m from
akos.palfi@imgtec.com
Link to the patchset: https://codereview.chromium.org/1519303002/#ps20001
(title: "store zero_reg, save an instruction.")

[commit-bot: I haz the power, 2015-12-12 03:12:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1519303002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1519303002/20001

[commit-bot: I haz the power, 2015-12-12 03:29:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-12 03:30:26]

Description was changed from

==========
MIPS: Fix for instanceof-proxies failure.

Invalidate the instanceof cache in proxy-case of InstanceOfStub
after the changes in https://codereview.chromium.org/1492243003.

This didn't have a test case until https://codereview.chromium.org/1518773003

TEST=mjsunit/es6/instanceof-proxies.js
BUG=
==========

to

==========
MIPS: Fix for instanceof-proxies failure.

Invalidate the instanceof cache in proxy-case of InstanceOfStub
after the changes in https://codereview.chromium.org/1492243003.

This didn't have a test case until https://codereview.chromium.org/1518773003

TEST=mjsunit/es6/instanceof-proxies.js
BUG=

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

[commit-bot: I haz the power, 2015-12-12 03:30:26]

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