Closed
Bug 31926
Opened 25 years ago
Closed 24 years ago
for() loops significantly slower than MSIE
Categories
(Core :: JavaScript Engine, defect, P3)
Tracking
()
VERIFIED
WORKSFORME
Future
People
(Reporter: warnckew, Assigned: rogerl)
Details
(Keywords: perf, testcase)
Attachments
(2 files)
I'll upload a test case for this, a simple for() loop that does exactly nothing,
but does it 10 million times. tested it in MSIE 5.01, Netscape Comm 4.72 and
Mozilla build 200031419. result was (all lapsed time in milliseconds):
Mozilla: 15100, 15100, 15210, 15110, 15100
IE: 8620, 8570, 8790, 8680, 8730
NC4.72: 12960, 13010, 13010, 13010, 13020
since I'm not the one to decide whether this is a significant problem or not,
I'll leave it up to you guys.
Reporter | ||
Comment 1•25 years ago
|
||
Reporter | ||
Comment 2•25 years ago
|
||
adding keyword 'testcase' as per bugathon guidelines
Keywords: testcase
hmm...interestingly I found mozilla was slightly faster than netscape 4.x. I
don't have IE though so I couldn't check that
cc'ing me. oh by the way the reporter doesn't need to cc him/herself as it
happens automatically :)
Reporter | ||
Comment 5•25 years ago
|
||
I figured out the Cc'ing thing after submitting another bug, tried to remove
myself again now. :)
downloaded build 200042608, and as b.judd found the speed has increased
somewhat. I clock in at around 12500ms now, which is around half a second
faster than NC4.72.
hmm I just tried 2000040708 and it's slower :(. I have no real idea how to test
performance bugs though. Someone needs to try profiling this beast and seeing
what's happening (not me though I can't build).
Comment 7•25 years ago
|
||
14000 in Moz, 16480 in NS for me. That's a PIII 450 :-)
Confirming, adding perf keyword.
Gerv
Comment 9•24 years ago
|
||
This is a dup of the fixed bug 43902. If you try your test again I think you'll
find the score much improved.
*** This bug has been marked as a duplicate of 43902 ***
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → DUPLICATE
Reporter | ||
Comment 10•24 years ago
|
||
Since I cannot verify any significant speed increase as of build 2000090508, I
reopen this bug. testing with the attached testcase returns scores in the
vicinity of 15000ms, give or take half a second. I do get two alerts about a
script slowing Mozilla down, and quickly dispatch these.
I am not sure if the problem is now associated with the alerts, or whatnot. a
for() loop with 1,000,000 iterations run at the same speed as my Communicator
4.75 (~1350ms).
I'm still able to duplicate the test results with IE, so we're still 7 seconds
behind.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment 12•24 years ago
|
||
Comment 13•24 years ago
|
||
I now get these timings for ten million iterations using the two
testcases above:
Mozilla binary 2001030310 WinNT : 10000 ms
In optimized JS shell on WinNT : 7900 ms
Since this represents a significant improvement over the figures
originally reported, I would like to close this as WORKSFORME.
Notice the Mozilla figure would be even lower if it weren't for
the two warning alertboxes one has to dismiss during the test.
Morten, could you confirm this with a recent build? If you agree,
please mark this bug "Verified"; otherwise, reopen it - thanks.
Status: REOPENED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Comment 14•23 years ago
|
||
I'm getting consistent <10s runs in the browser testcase with build 2001052204,
therefore I mark it verified. thanks! :)
Status: RESOLVED → VERIFIED
Comment 15•23 years ago
|
||
Anybody happen to know if there is a bug filed already for the fact that the
alert comes up more than once for the same script? I would think it ought to
remember if you say "No"
You need to log in
before you can comment on or make changes to this bug.
Description
•