This enables the check for GC'ing sections when compiling with GCC and GNU ld. GC'ing sections saves ~1.5MB off of libxul according to size(1): [froydnj@cerebro eideticker]$ size /opt/build/froydnj/build-android/dist/bin/libxul.so text data bss dec hex filename 23733400 2314816 1495973 27544189 1a44a7d /opt/build/froydnj/build-android/dist/bin/libxul.so [froydnj@cerebro eideticker]$ size /opt/build/froydnj/build-android/dist/bin/libxul.so text data bss dec hex filename 22261936 2145116 1494533 25901585 18b3a11 /opt/build/froydnj/build-android/dist/bin/libxul.so

I don't know that this is the best way to fix things, but it at least makes android work the same as Linux. Tested with GCC 4.6.

Comment on attachment 768436 [details] [diff] [review] set MOZ_DEBUG_FLAGS for android targets Review of attachment 768436 [details] [diff] [review]: ----------------------------------------------------------------- The right thing to do here is to move the setting and resetting of MOZ_DEBUG_FLAGS above MOZ_COMPILER_OPTS. Which feels like an old tune, I think there was a bug about that already.

MOZ_COMPILER_OPTS depends on MOZ_DEBUG_FLAGS being set for proper checking of linker dead code deletion when using GCC. I think this DTRT, currently running on try: https://tbpl.mozilla.org/?tree=Try&rev=cd407a2d4067

Sigh, check-sync-dirs. https://tbpl.mozilla.org/?tree=Try&rev=7f8a626528e7

Comment on attachment 770241 [details] [diff] [review] move --enable-debug{,-symbols} option processing before MOZ_COMPILER_OPTS Review of attachment 770241 [details] [diff] [review]: ----------------------------------------------------------------- Let's hope it doesn't have bad side effects. ::: configure.in @@ +2542,5 @@ > +dnl ======================================================== > +dnl = Debug info is ON by default. > +dnl ======================================================== > +if test -z "$MOZ_DEBUG_FLAGS"; then > + MOZ_DEBUG_FLAGS="-g" Considering this, you can also remove any MOZ_DEBUG_FLAGS=-g above.

Might as well do the same thing for the JS engine.

Comment on attachment 771276 [details] [diff] [review] move --enable-debug{,-symbols} option processing before MOZ_COMPILER_OPTS, js version Review of attachment 771276 [details] [diff] [review]: ----------------------------------------------------------------- Hoping this will stick.

(In reply to Mike Hommey [:glandium] from comment #7) > Hoping this will stick. Inbound is the new try, right? ;) http://hg.mozilla.org/integration/mozilla-inbound/rev/6fe449ecd692

According to http://areweslimyet.com/mobile this reduces startup memory usage by ~2.5MB, which is a nice win!