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

PPC: 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.

R=mbrandy@us.ibm.com, svenpanne@chromium.org

BUG=

Committed: https://crrev.com/039247c4f347ab80a1f4cd5dab0cf7877eb08625
Cr-Commit-Position: refs/heads/master@{#27430}

[michael_dawson, 2015-03-24 16:47:48]

Latest PPC port.

Seven,could you add your approval to this one and the previous one, I think PPC
should compile/pass tests after these two get in.

[MTBrandyberry, 2015-03-24 16:58:48]

On 2015/03/24 16:47:48, michael_dawson wrote:
> Latest PPC port.
> 
> Seven,could you add your approval to this one and the previous one, I think
PPC
> should compile/pass tests after these two get in.

lgtm

[michael_dawson, 2015-03-25 01:50:40]

The CQ bit was checked by michael_dawson@ca.ibm.com

[commit-bot: I haz the power, 2015-03-25 01:51:23]

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

[commit-bot: I haz the power, 2015-03-25 06:42:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-25 06:42:30]

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