Optimize GetPrototype

Optimize GetPrototype

This introduces Hydrogen for %_GetPrototype. The code falls back on
runtime if the object needs access checks or if its prototype is a
hidden prototype.

BUG=None
LOG=Y
R=dslomov@chromium.org

[arv (Not doing code reviews), 2014-12-05 15:15:03]

arv@chromium.org changed reviewers:
+ verwaest@chromium.org

[arv (Not doing code reviews), 2014-12-05 15:15:04]

Motivation: I want to use GetPrototype in the default constructor for classes.

[arv (Not doing code reviews), 2014-12-05 15:29:13]

Skip push/pop

[arv (Not doing code reviews), 2014-12-05 15:30:05]

PTAL

https://codereview.chromium.org/756423006/diff/20001/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/756423006/diff/20001/src/hydrogen.cc#newcode1...
src/hydrogen.cc:12432: ast_context()->ReturnValue(result);
Other places usually do a push/pop in this case but it seems cleaner to skip
these extra instructions.

[Dmitry Lomov (no reviews), 2014-12-05 15:30:28]

excellent, looks good, will also come handy for enabling hydrogen for
'super(...)' calls.
This need a test: here is some code for inspiration:
https://code.google.com/p/chromium/codesearch#chromium/src/v8/test/cctest/tes...

In test-api.cc, you can just set FLAG_allow_natives_syntax = true, and then
validate that the script throws:

   function f(o) { return %_GetPrototype(o); }
   %OptimizeFunctionOnNextCall(f);
   f(prohibited)

[Dmitry Lomov (no reviews), 2014-12-05 15:39:20]

Nit

https://codereview.chromium.org/756423006/diff/20001/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/756423006/diff/20001/src/hydrogen.cc#newcode1...
src/hydrogen.cc:12432: ast_context()->ReturnValue(result);
On 2014/12/05 15:30:05, arv wrote:
> Other places usually do a push/pop in this case but it seems cleaner to skip
> these extra instructions.

Nit: could you keep the push/pop code pattern? This makes it very clear that
only one value is ever returned from this Visit method. (Push/Pop has no effect
on resulting codegen)

[arv (Not doing code reviews), 2014-12-05 16:45:24]

Add test and add back Push/Pop

[arv (Not doing code reviews), 2014-12-05 16:46:09]

PTAL

[Dmitry Lomov (no reviews), 2014-12-05 20:04:03]

lgtm, thanks!

[arv (Not doing code reviews), 2014-12-05 20:09:55]

The CQ bit was checked by arv@chromium.org

[commit-bot: I haz the power, 2014-12-05 20:10:14]

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

[commit-bot: I haz the power, 2014-12-05 20:36:47]

Committed patchset #3 (id:40001)