[v8-dev] Re: Direct call accessor getter callbacks (arm implementation). (issue6462029)

2011-02-24 Thread antonm
Thanks a lot, Zaheer. Landing it. http://codereview.chromium.org/6462029/ -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev

[v8-dev] Re: Direct call accessor getter callbacks (arm implementation). (issue6462029)

2011-02-22 Thread zahmad
hi Anton, Addressed comments. could you take another look. Thanks. http://codereview.chromium.org/6462029/diff/1/src/arm/stub-cache-arm.cc File src/arm/stub-cache-arm.cc (right): http://codereview.chromium.org/6462029/diff/1/src/arm/stub-cache-arm.cc#newcode1245 src/arm/stub-cache-arm.cc:1245:

[v8-dev] Re: Direct call accessor getter callbacks (arm implementation). (issue6462029)

2011-02-21 Thread antonm
Almost LGTM, let's do last round and I'll land it http://codereview.chromium.org/6462029/diff/1/src/arm/simulator-arm.cc File src/arm/simulator-arm.cc (right): http://codereview.chromium.org/6462029/diff/1/src/arm/simulator-arm.cc#newcode1534 src/arm/simulator-arm.cc:1534: typedef v8::Handle (*

[v8-dev] Re: Direct call accessor getter callbacks (arm implementation). (issue6462029)

2011-02-21 Thread zahmad
Thanks Anton, Serya for your comments. Please review the latest patch. http://codereview.chromium.org/6462029/diff/1/src/arm/simulator-arm.cc File src/arm/simulator-arm.cc (right): http://codereview.chromium.org/6462029/diff/1/src/arm/simulator-arm.cc#newcode1534 src/arm/simulator-arm.cc:1534:

[v8-dev] Re: Direct call accessor getter callbacks (arm implementation). (issue6462029)

2011-02-20 Thread serya
http://codereview.chromium.org/6462029/diff/1/src/arm/stub-cache-arm.cc File src/arm/stub-cache-arm.cc (right): http://codereview.chromium.org/6462029/diff/1/src/arm/stub-cache-arm.cc#newcode1260 src/arm/stub-cache-arm.cc:1260: __ push(name_reg); __ Push(receiver, scratch3, name_reg) (or at leas

[v8-dev] Re: Direct call accessor getter callbacks (arm implementation). (issue6462029)

2011-02-17 Thread antonm
Thanks a lot for working on this! http://codereview.chromium.org/6462029/diff/1/src/arm/simulator-arm.cc File src/arm/simulator-arm.cc (right): http://codereview.chromium.org/6462029/diff/1/src/arm/simulator-arm.cc#newcode1534 src/arm/simulator-arm.cc:1534: typedef v8::Handle (*SimulatorRuntime