Bug 21463 - 11.8% SunSpider regression in non-CTI from r37324
Summary: 11.8% SunSpider regression in non-CTI from r37324
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-08 01:54 PDT by Gabor Loki
Modified: 2010-05-25 13:52 PDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Gabor Loki 2008-10-08 01:54:52 PDT
Hi,

we have noticed a 11.8% slow down on SunSpider from 37324 revision on Qt-linux port (it was 15.7% regression on V8).

Do you know this? Or does it appear only on linux?

You can also check our results from http://www.sed.hu/webkit/?page=timeline or http://www.sed.hu/webkit/?page=revisions&order=ASC&num=30&pg=26 pages.
Comment 1 Cameron Zwarich (cpst) 2008-10-08 14:27:12 PDT
This is in bytecode, not CTI, right?
Comment 2 Gabor Loki 2008-10-09 00:58:46 PDT
You are right. This happened in bytecode (CTI disabled).

As far as I know CTI is only enabled for Windows and Mac on x86 target.
Platform.h says: #if !defined(ENABLE_CTI) && PLATFORM(X86) && (PLATFORM(MAC) || PLATFORM(WIN))

Probably, I can check if it exists on qt-linux with CTI.
Comment 3 Cameron Zwarich (cpst) 2008-10-09 01:04:20 PDT
Some of the Qt people have a branch based on the patch to enable CTI on Linux. I was wondering if you were using that.

Since this is my patch, I will check what happens on the Mac tomorrow.
Comment 4 Gabor Loki 2008-10-09 01:24:25 PDT
Do you mean code.staikos.net? No, I did not use it yet, and I have not found any CTI related modification in that tree.

We have a separate patch to enable CTI for Qt port. I will send it as soon as possible.
Comment 5 Gabor Loki 2008-11-10 01:52:09 PST
(In reply to comment #3)
> Since this is my patch, I will check what happens on the Mac tomorrow.

Cameron, is there any progress in this issue?

You can check our performance results at:
http://www.sed.hu/webkit/?page=timelinerange&s_ss=on&s_ws=on&s_v8=on&from=37301&till=37396
Comment 6 Cameron Zwarich (cpst) 2008-11-10 02:02:40 PST
I'll try to take a look at it today on my machine. I suspect that it is something about your compiler version or flags, because no one tracking PPC performance has mentioned such a dramatic drop.
Comment 7 Cameron Zwarich (cpst) 2008-11-10 02:16:57 PST
I see no slowdown between those two revisions on my machine, and I actually see a slight speedup.
Comment 8 Gabor Loki 2008-11-10 02:24:34 PST
No regression? On Mac, am I right?
It would be nice if someone else can confirm or reject it on linux!

Anyway, this is not the only one regression over 10%.
Between 38153 and 38187: http://www.sed.hu/webkit/?page=timelinerange&s_ss=on&s_ws=on&s_v8=on&from=38114&till=38201

I will open another bug report if I find the concert changeset number.
Comment 9 Cameron Zwarich (cpst) 2008-11-10 02:31:49 PST
I am on Mac. Have you tried using the same compiler flags for optimizations as the Mac port, and using an AllInOne.cpp build?
Comment 10 Gabor Loki 2008-11-10 02:44:34 PST
Not yet.
We are using the 'WebKitTools/Scripts/build-webkit --qt' script which uses
DEFINES = -DBUILDING_QT__ -DNDEBUG -DUSE_SYSTEM_MALLOC -DBUILDING_QT__=1 -DUSE_SYSTEM_MALLOC -DNDEBUG -DBUILDING_QT__ -DQT_NO_DEBUG -DQT_CORE_LIB -DQT_SHARED
and
CFLAGS = -pipe -O2 -Wall -W -D_REENTRANT $(DEFINES)

without AllInOneFile.cpp method

As I see in the Makefiles neither GTK nor QT-port uses AllInOneFile.cpp feature.
Well, GTK uses AllInOneFile.cpp, but only in Debug mode.
Comment 11 Alexey Proskuryakov 2010-05-25 13:52:36 PDT
This bug seems too old to be useful. Please feel free to re-open if this is still an issue.