Vector-based LoadICs need to handle Smi receivers.

Vector-based LoadICs need to handle Smi receivers.

The MISS handler was being called when the receiver was a Smi, instead,
we should recognize the case and use the heap number map.

BUG=

Committed: https://crrev.com/23b181552e63aa3a6ee86ab0c99998dad44b7f14
Cr-Commit-Position: refs/heads/master@{#26076}

[mvstanton, 2015-01-14 16:25:01]

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

[mvstanton, 2015-01-14 16:25:01]

Hi Yang, here is the change we discussed. Really helps Octane PdfJs (from -30%
to -4%, w00t).

[Yang, 2015-01-15 11:33:46]

lgtm with nit.

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

https://codereview.chromium.org/854623002/diff/1/src/code-stubs-hydrogen.cc#n...
src/code-stubs-hydrogen.cc:2042: Push(AddLoadMap(receiver, nullptr));
Don't we usually just use NULL (instead of nullptr)?

[mvstanton, 2015-01-15 12:25:39]

Thanks Yang, landing...

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

https://codereview.chromium.org/854623002/diff/1/src/code-stubs-hydrogen.cc#n...
src/code-stubs-hydrogen.cc:2042: Push(AddLoadMap(receiver, nullptr));
On 2015/01/15 11:33:45, Yang wrote:
> Don't we usually just use NULL (instead of nullptr)?

Looks like someone did a refactoring CL a while back that changed this. I
believe the NULLs had a static_cast around them, and this was cleaner. In the
rest of the file we've got "nullptr" for this situation, so I'll leave it.

[mvstanton, 2015-01-15 12:25:46]

The CQ bit was checked by mvstanton@chromium.org

[commit-bot: I haz the power, 2015-01-15 12:26:06]

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

[commit-bot: I haz the power, 2015-01-15 12:52:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-01-15 12:52:59]

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