Please take another look.
https://codereview.chromium.org/1146963002/diff/20001/src/compiler/js-intrinsic-lowering.cc
File src/compiler/js-intrinsic-lowering.cc (right):
https://codereview.chromium.org/1146963002/diff/20001/src/compiler/js-intrinsic-lowering.cc#newcode470
src/compiler/js-intrinsic-lowering.cc:470:
NodeProperties::ReplaceWithValue(node, outer_frame, effect, control);
On 2015/05/20 14:25:18, Michael Starzinger wrote:
nit: Please use AdvancedReducer::ReplaceWithValue here, I am currently
switching
the reducers over to this method, the old
NodeProperties::ReplaceWithValue is
deprecated.
Done.
https://codereview.chromium.org/1146963002/diff/20001/src/compiler/js-intrinsic-lowering.cc#newcode486
src/compiler/js-intrinsic-lowering.cc:486: access =
AccessBuilder::ForFrameMarker();
On 2015/05/20 14:25:18, Michael Starzinger wrote:
Does this work with arguments adapter frames and construct frames? Can
we have
tests for those?
Now it does :-)
https://codereview.chromium.org/1146963002/diff/20001/src/compiler/simplified-lowering.cc
File src/compiler/simplified-lowering.cc (right):
https://codereview.chromium.org/1146963002/diff/20001/src/compiler/simplified-lowering.cc#newcode1045
src/compiler/simplified-lowering.cc:1045: case
IrOpcode::kLoadFramePointer:
On 2015/05/20 14:25:18, Michael Starzinger wrote:
nit: Lets just have both labels (i.e. kLoadStackPointer and
kLoadFramePointer)
point to one single return statement.
Done.
https://codereview.chromium.org/1146963002/diff/20001/test/mjsunit/get-caller-js-function.js
File test/mjsunit/get-caller-js-function.js (right):
https://codereview.chromium.org/1146963002/diff/20001/test/mjsunit/get-caller-js-function.js#newcode8
test/mjsunit/get-caller-js-function.js:8: return %GetCallerJSFunction();
On 2015/05/20 14:05:48, Sven Panne wrote:
This will *always* call the runtime function, do you mean
%_GetCallerJSFunction?
Good catch. Will fix.
https://codereview.chromium.org/1146963002/diff/20001/test/mjsunit/get-caller-js-function.js#newcode18
test/mjsunit/get-caller-js-function.js:18: c();
On 2015/05/20 14:25:18, Michael Starzinger wrote:
nit: Can we wrap all three calls to "c" into an assertEquals checking
the return
value?
Done.
https://codereview.chromium.org/1146963002/diff/20001/test/mjsunit/get-caller-js-function.js#newcode28
test/mjsunit/get-caller-js-function.js:28: d();
On 2015/05/20 14:25:18, Michael Starzinger wrote:
nit: Likewise.
Done.
https://codereview.chromium.org/1146963002/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
