[v8-dev] Re: Re-submitting binary op ICs for ARM. Does not break debug tests (issue1629008)

2010-04-08 Thread Mads Sig Ager
Thanks Vlad. Still LGTM. :) -- Mads On Thu, Apr 8, 2010 at 5:34 PM, Vladislav Kaznacheev wrote: > Sure. On ia32 and x64 there is a special branch for handling a pair of > smis (GenericBinaryOpStub::GenerateSmiCode), so a pair of smis never > triggers a transition. On ARM there is a single code p

[v8-dev] Re: Re-submitting binary op ICs for ARM. Does not break debug tests (issue1629008)

2010-04-08 Thread Vladislav Kaznacheev
Sure. On ia32 and x64 there is a special branch for handling a pair of smis (GenericBinaryOpStub::GenerateSmiCode), so a pair of smis never triggers a transition. On ARM there is a single code path for all combinations of operand times so a pair of smis can actually be passed to the method in quest

[v8-dev] Re: Re-submitting binary op ICs for ARM. Does not break debug tests (issue1629008)

2010-04-08 Thread Mads Sig Ager
Can you quickly explain why that assert is triggered on ARM and not on the other platforms? Thanks,-- Mads On Thu, Apr 8, 2010 at 5:17 PM, Vladislav Kaznacheev wrote: > That and a change in BinaryOpIC::GetTypeInfo that also have been > breaking debug tests. > > On Thu, Apr 8, 2010 at 7:12 PM

[v8-dev] Re: Re-submitting binary op ICs for ARM. Does not break debug tests (issue1629008)

2010-04-08 Thread Vladislav Kaznacheev
That and a change in BinaryOpIC::GetTypeInfo that also have been breaking debug tests. On Thu, Apr 8, 2010 at 7:12 PM, wrote: > > LGTM, I suppose the only change to the previous submit is the removal of the > clearing on GC? > > http://codereview.chromium.org/1629008/show -- v8-dev mailing list

[v8-dev] Re: Re-submitting binary op ICs for ARM. Does not break debug tests (issue1629008)

2010-04-08 Thread ager
LGTM, I suppose the only change to the previous submit is the removal of the clearing on GC? http://codereview.chromium.org/1629008/show -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev To unsubscribe, reply using "remove me" as the subject.