Hi Tom,
On Tue, Aug 7, 2012 at 1:11 AM, Ilya Yanok ilya.yanok@cogentembedded.comwrote:
Yes. What I meant was that not all of the stuff that is guarded today
is garbage collected so the resulting image is larger than it must be.
Yep. And that's actually goes beyond the subject of this patch: as long as we rely on linker GC and not putting guards by hands we will get unused string literals compiled in.
I still think that this patch does the correct things (UNDEF_SYM is there to protect commands from GC and we don't really want commands in SPL so there is nothing to protect). But I will try to add guards to make net-spl compile even without this patch.
I've just posted v5 for patch 4/5 that doesn't depend on this patch. It really has some improvement wrt size (~50KB net-only SPL image) so probably it could be useful not only for AM33xx...
Regards, Ilya.