From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Mon, 31 Oct 2011 18:37:32 -0400 Subject: [U-Boot] [PATCH] cmd_bdinfo: simplify local static funcs a bit In-Reply-To: References: <1320022463-26410-1-git-send-email-vapier@gentoo.org> Message-ID: <201110311837.33466.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Monday 31 October 2011 17:49:58 Simon Glass wrote: > On Mon, Oct 31, 2011 at 2:15 PM, Tabi Timur-B04825 wrote: > > On Sun, Oct 30, 2011 at 7:54 PM, Mike Frysinger wrote: > >> -static void print_num(const char *, ulong); > >> +__maybe_unused > >> +static void print_num(const char *name, ulong value) > >> +{ > >> + printf("%-12s= 0x%08lX\n", name, value); > >> +} > > > > Will the linker remove the functions from the binary if they are unusued? > > If built with -ffunction-sections and --gc-sections,, then the linker > can do this sort of thing. Otherwise it can't, but the compiler can. I > just tested Mike's code on my ARM compiler to make sure and it happily > removed print_eth() when it was not used. i don't think you need function-sections to make this happen. since it is marked "static", gcc should do DCE on it. the __maybe_unused markings is just to kill off any warnings about the func not being used (which might occur in the #ifdef jungle below). that attribute does not tell gcc to retain the function even if it isn't referenced in this file (as far as gcc can tell). -mike -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: This is a digitally signed message part. Url : http://lists.denx.de/pipermail/u-boot/attachments/20111031/233f254c/attachment.pgp