[2.6 patch] offer CC_OPTIMIZE_FOR_SIZE only if EXPERIMENTAL

From: Adrian Bunk <bunk@stusta.de>
To: Andrew Morton <akpm@osdl.org>
Cc: torvalds@osdl.org, linux-kernel@vger.kernel.org
Subject: [2.6 patch] offer CC_OPTIMIZE_FOR_SIZE only if EXPERIMENTAL
Date: Wed, 14 Dec 2005 23:13:04 +0100	[thread overview]
Message-ID: <20051214221304.GE23349@stusta.de> (raw)
In-Reply-To: <20051214140531.7614152d.akpm@osdl.org>

On Wed, Dec 14, 2005 at 02:05:31PM -0800, Andrew Morton wrote:
> Adrian Bunk <bunk@stusta.de> wrote:
> >
> > Hi Linus,
> > 
> > your patch to allow CC_OPTIMIZE_FOR_SIZE even for EMBEDDED=n has broken 
> > the EMBEDDED menu.
> 
> It looks like that patch needs to be reverted or altered anyway.  sparc64
> machines are failing all over the place, possibly due to newly-exposed
> compiler bugs.
> 
> Whether it's the compiler or it's genuine kernel bugs, the same problems
> are likely to bite other architectures.

The help text already contains a bold warning.

What about marking it as EXPERIMENTAL?
That is not that heavy as EMBEDDED but expresses this.

cu
Adrian


<--  snip  -->


CC_OPTIMIZE_FOR_SIZE is still an experimental feature that doesn't work 
with all supported gcc/architecture combinations.


Signed-off-by: Adrian Bunk <bunk@stusta.de>

--- linux-git/init/Kconfig.old	2005-12-14 23:08:51.000000000 +0100
+++ linux-git/init/Kconfig	2005-12-14 23:09:09.000000000 +0100
@@ -257,7 +257,7 @@
 source "usr/Kconfig"
 
 config CC_OPTIMIZE_FOR_SIZE
-	bool "Optimize for size"
+	bool "Optimize for size (EXPERIMENTAL)" if EXPERIMENTAL
 	default y if ARM || H8300
 	help
 	  Enabling this option will pass "-Os" instead of "-O2" to gcc

