MIPS64: VectorICs: keyed element loads were kicking out non-smi keys unnecessarily

MIPS64: VectorICs: keyed element loads were kicking out non-smi keys unnecessarily

Port 6689cc27ebe60685c025de9ae1f09919093f8213

Original commit message:
Handlers should be in charge of this work. The change uncovered a bug in
vector-ics related to keyed loads into strings. It's important for
StringCharCodeAtGenerator, a helper used in full code and in
LoadIndexedStringStub (a handler) to protect the vector and slot registers
when it makes a runtime call to convert a HeapNumber to a Smi.

It's still possible for the handler to MISS after this call, perhaps due
to out of bounds access. In that case, the vector and slot registers need
to be delivered safely to the MISS handler.

BUG=

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

[akos.palfi.imgtec, 2015-03-23 20:58:47]

akos.palfi@imgtec.com changed reviewers:
+ balazs.kilvady@imgtec.com, danno@chromium.org, dusan.milosavljevic@imgtec.com,
gergely.kis@imgtec.com, mvstanton@chromium.org, paul.lind@imgtec.com

[akos.palfi.imgtec, 2015-03-23 20:58:47]

PTAL.

[mvstanton, 2015-03-23 21:04:13]

LGTM, thanks for the port!

[paul.l..., 2015-03-23 21:06:38]

Also LGTM. Thanks for the review & the mips32 port Michael!

[akos.palfi.imgtec, 2015-03-23 21:42:23]

The CQ bit was checked by akos.palfi@imgtec.com

[commit-bot: I haz the power, 2015-03-23 21:42:39]

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

[commit-bot: I haz the power, 2015-03-23 22:45:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-23 22:45:28]

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