Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S265097AbTFMBGx (ORCPT ); Thu, 12 Jun 2003 21:06:53 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S265092AbTFMBFo (ORCPT ); Thu, 12 Jun 2003 21:05:44 -0400 Received: from dp.samba.org ([66.70.73.150]:26853 "EHLO lists.samba.org") by vger.kernel.org with ESMTP id S265089AbTFMBFU (ORCPT ); Thu, 12 Jun 2003 21:05:20 -0400 From: Rusty Russell To: torvalds@transmeta.com Cc: akpm@zip.com.au, linux-kernel@vger.kernel.org, Kai Germaschewski Subject: Re: [PATCH] Make gcc3.3 Eliminate Unused Static Functions In-reply-to: Your message of "Fri, 13 Jun 2003 10:17:05 +1000." Date: Fri, 13 Jun 2003 11:03:43 +1000 Message-Id: <20030613011906.8BAD92C23B@lists.samba.org> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2715 Lines: 63 Argh, bogus line pasted into Makefile turned up in patch. This should be better... Name: Eliminate Unused Functions Author: Rusty Russell Status: Tested on 2.5.70-bk16 D: GCC 3.3 has the ability to eliminate unused static functions. This includes D: code like this: D: D: static int unusedfunc(void) { ... }; D: int otherfunc(void) D: { D: (void)unusedfunc; D: ... D: D: This means that macros can suppress the "unused" warning on functions D: without preventing the function elimination. This should allow us to D: remove a number of #ifdefs around unused functions. D: D: Unfortunately, this elimination is only performed if D: -finline-functions is used. In order to prevent GCC automatically D: inlining anything, we also specify "--param max-inline-insns-auto=0". D: D: Earlier compilers don't understand this parameter, so we test for D: it at build time. D: D: Results: D: gcc 3.3 without patch: D: -rwxrwxr-x 1 rusty rusty 5115166 Jun 13 09:17 vmlinux D: gcc 3.3 with patch: D: -rwxrwxr-x 1 rusty rusty 5115166 Jun 13 09:58 vmlinux D: gcc 3.3 without patch (small unused function added): D: -rwxrwxr-x 1 rusty rusty 5115195 Jun 13 10:14 vmlinux D: gcc 3.3 with patch (small unused function added): D: -rwxrwxr-x 1 rusty rusty 5115166 Jun 13 10:15 vmlinux diff -urpN --exclude TAGS -X /home/rusty/devel/kernel/kernel-patches/current-dontdiff --minimal working-2.5.70-bk16-check_region/Makefile working-2.5.70-bk16-check_region-inline/Makefile --- working-2.5.70-bk16-check_region/Makefile 2003-06-12 09:57:39.000000000 +1000 +++ working-2.5.70-bk16-check_region-inline/Makefile 2003-06-12 21:34:40.000000000 +1000 @@ -213,10 +213,12 @@ CFLAGS_KERNEL = AFLAGS_KERNEL = NOSTDINC_FLAGS = -nostdinc -iwithprefix include +# Needs gcc 3.3 or above to understand max-inline-insns-auto. +INLINE_OPTS := $(shell $(CC) -o /non/existent/file -c --param max-inline-insns-auto=0 -xc /dev/null 2>&1 | grep /non/existent/file >/dev/null && echo -finline-functions --param max-inline-insns-auto=0) CPPFLAGS := -D__KERNEL__ -Iinclude CFLAGS := $(CPPFLAGS) -Wall -Wstrict-prototypes -Wno-trigraphs -O2 \ - -fno-strict-aliasing -fno-common + $(INLINE_OPTS) -fno-strict-aliasing -fno-common AFLAGS := -D__ASSEMBLY__ $(CPPFLAGS) export VERSION PATCHLEVEL SUBLEVEL EXTRAVERSION KERNELRELEASE ARCH \ -- Anyone who quotes me in their sig is an idiot. -- Rusty Russell. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/