Reviewers: antonm, Kevin Millikin,

Message:
Cleanup ready to commit.


http://codereview.chromium.org/604054/diff/1/2
File src/arm/stub-cache-arm.cc (right):

http://codereview.chromium.org/604054/diff/1/2#newcode1055
src/arm/stub-cache-arm.cc:1055: <<<<<<< .mine
On 2010/02/15 08:47:37, antonm wrote:
merge conflict

Done.

http://codereview.chromium.org/604054/diff/1/2#newcode1065
src/arm/stub-cache-arm.cc:1065: CallInterceptorCompiler
compiler(arguments());
On 2010/02/15 08:47:37, antonm wrote:
1) probably the whole compiler should go away as it now only have one
method;
2) I am slightly concerned that various platform now diverge---could
we bring
the similar change to other platforms?

The compiler class has gone away.

We could bring the change to other platforms, when there is time.

Description:
Simplify compilation of CallIC interceptor stub.

Please review this at http://codereview.chromium.org/604054

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/arm/stub-cache-arm.cc


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to