New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 702662 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Change build params on libxml to save space (24kb)

Project Member Reported by dtapu...@chromium.org, Mar 17 2017

Issue description

Try adjusting the libxml build parameters and you'll save memory.

I believe you can turn off LIBXML_LEGACY_ENABLED, LIBXML_DEBUG_ENABLED

Turning these off with net roughly 12K

You might get away dropping LIB_XPTR_ENABLED which will be another 12K.

 
Labels: Binary-Size
Summary: Change build params on libxml to save space (24kb) (was: Change build params on libxml to save space)
Thanks Dave, this is great!
Cc: agrieve@chromium.org
Owner: dominicc@chromium.org
Dominic - saw you touched this recently (https://codereview.chromium.org/2789473002).

Wonder if you'd be willing to look at tweaking these defines as well?
Cc: dcheng@chromium.org
Status: Started (was: Assigned)
Thanks for routing this to me; I turned off a bunch of these including LEGACY, XPTR, DEBUG and others in r461657.

dcheng is making some changes to other libxml2 users. I believe updater is the last user of SAX1 and WRITER, so there may be a bit more which can be wrung out here.
Status: Fixed (was: Started)
I'm going to mark this as fixed because we've probably hit a local optima with this. LEGACY, DEBUG, XPTR and various others are gone in M58-59; hopefully that will stick.

Could you verify that did indeed shrink things?

Just curious but what's the easiest way to spelunk sizes like this these days?
Looks like r461657 reduced size by ~32800 bytes:
https://chromeperf.appspot.com/report?sid=ec464e89358810af9b857af88d316e5ee1b735fc532f3b1a883380d26d29da06&rev=461657

Best way to look at size is the graph I linked, as well as using tools at //tools/binary_size.
Labels: -binary-size Performance-Size

Sign in to add a comment