Bug 24891 – WebCore can be 600K smaller

• Component: New Bugs
• Status: RESOLVED WONTFIX

[Greg Bolsinga]

With exported C++ and Obj-C++ files in WebCore and WebKit, you'll see the C++ namespaces strings repeated often in the binaries.

By renaming the namespaces (WebCore -> W, WTF -> F, JSC -> J) the total size of the 3 binaries can be reduced by ~600K, in my testing.

This is extremely advantageous on memory constrained platforms. You can see some Obj-C++ strings in WebCore (via strings - <WebCoreLibrary>| grep WebCore) that are extremely long. These affect performance.

We have learned that all of these strings are touched during library use. The smaller the library, and the shorter the strings the better on memory constrained platforms.

This patch fixes this in a ham-handed way (#defines at in Platform.h) and post-processes the .exp files, but it is a proof of concept.

Our performance team indicates this is a big win on our platform.

[Greg Bolsinga]

Created attachment 29021 [details]
Patch makes WebCore ~600K smaller

This is a ham-handed patch so that this can be tested out for performance testing.

[Greg Bolsinga]

And by indicates  I mean believes. Testing is currently in progress.

[Alexey Proskuryakov]

I would expect memory-constrained platforms to just strip the binary of all method names, for a much bigger win. 

[David Kilzer (:ddkilzer)]

(In reply to comment #3)
> I would expect memory-constrained platforms to just strip the binary of all
> method names, for a much bigger win. 

Objective-C method names can't be stripped, though, and some selectors in Objective-C++ source code (which include C++-mangled type names) are multiple *kilobytes*--for a single selector.  That's where the benefit lies for this change.

[Greg Bolsinga]

There are small size wins in JavaScriptCore as well, so it isn't just all Obj-C++.

[Greg Bolsinga]

<rdar://problem/6729586>

[Greg Bolsinga]

Here's some bugs that address this problem at a better level:

https://bugs.webkit.org/show_bug.cgi?id=25210
https://bugs.webkit.org/show_bug.cgi?id=25266

[David Kilzer (:ddkilzer)]

How much is saved after fixing Bug 25210?  Is this still worth doing?

[Eric Seidel (no email)]

Comment on attachment 29021 [details]
Patch makes WebCore ~600K smaller

The patch looks great!  Given your own reluctance in your last comment, I'm marking this r- for now.  Please feel encouraged to re-mark this r=? if this should actually still be reviewed.

[Greg Bolsinga]

I think the other bugs are better fixes for this, and iPhone already had a big win from Bug 25210. So I'm closing this as NTBF.