From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sam Ravnborg Subject: Re: [PATCH] kbuild: Disallow GCC 4.1.0 / 4.1.1 Date: Fri, 2 Jan 2009 18:57:37 +0100 Message-ID: <20090102175737.GA5818@uranus.ravnborg.org> References: <20081231105425.9ccac21d.akpm@linux-foundation.org> <20081231.141408.60544902.davem@davemloft.net> <20090102115547.GB3027@elte.hu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Ingo Molnar , David Miller , akpm@linux-foundation.org, rdreier@cisco.com, ian.campbell@citrix.com, jeremy.fitzhardinge@citrix.com, deller@gmx.de, rusty@rustcorp.com.au, linux-parisc@vger.kernel.org, linux-kernel@vger.kernel.org, kyle@mcmartin.ca, randolph@tausq.org, dave@hiauly1.hia.nrc.ca, Andrew Morton To: Linus Torvalds Return-path: In-Reply-To: List-ID: List-Id: linux-parisc.vger.kernel.org On Fri, Jan 02, 2009 at 09:38:14AM -0800, Linus Torvalds wrote: > > > On Fri, 2 Jan 2009, Linus Torvalds wrote: > > > > I think this is wrong. > > > > Just move the check into > > In fact, looking at that whole mess, I redid it all. It was disgusting how > conditionals in gcc4.h needed to double-check that __GNUC__ really was 4 > (rather than something bigger), which largely negated the whole nice clean > compiler version separation. > > I pushed out my preferred version, which fixes up the whole thing. The > gcc4 header file now only gets included for __GNUC__ == 4, and when we > ever see a __GNUC__ of 5, it will automatically DTRT and try to include > instead of #4. > > And then the check in gcc4.h for 4.1.0 and 4.1.1 is much simplified. The > two patches I pushed out add functionality, but don't actually add any new > lines (the first simplification patch removes more lines than it adds, and > the second one that adds the __weak bug test adds as mahy lines as the > cleanup removed). > > And it all looks more logical too, imho. Bugger.... Now I cannot do cross compile for: alpha, arm, m68k and sparc. Not that I actually try to run these beasts but just being able to do cross compile has served me well. Last I looked (only few days ago) crosstool-ng did not support sparc :-( Sam