913752 - Significant regression in jQuery performance from ion exceptions fix

Status: RESOLVED DUPLICATE of bug 914478

(Core :: JavaScript Engine, defect)

Description

[Boris Zbarsky [:bzbarsky]]

See bug 909764 comment 37.  We just regressed jQuery stuff by 15% or so.

Comment 1

[Kannan Vijayan [:djvj]]

This is probably due to changes in inlining behaviour in certain cases.  None of the other changes should have any performance effect.

The other changes all preserve the order and meaning of all calls, only adding the ability to bubble up an error to certain parts of the compiler.

Comment 2

[Kannan Vijayan [:djvj]]

Attachment: fix-regression.patch

Wanted to test this out for fixing the perf regression, but I'm in the middle of some flights.  bz, if you get time and are able to check this out, let me know.  I'll test it out next week.

Comment 3

[Boris Zbarsky [:bzbarsky]]

I'm not going to get a chance to look at this in the near future.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Bug 914478 might have fixed this.

Comment 5

[Kannan Vijayan [:djvj]]

I think it did.  I'll mark as duplicate once I confirm this more directly than just intuition.