Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread Eric Botcazou
If you configure a biarch Linux/Sparc compiler defaulting to 32-bit, but give --with-cpu= for a v9 cpu it erroneously turns on 64-bit in TARGET_DEFAULT. PR target/50354 reports the breakage of the opposite case after the change: configuring for sparc64-linux --with-cpu=v8 used to build a

Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread David Miller
From: Eric Botcazou ebotca...@adacore.com Date: Fri, 14 Oct 2011 20:27:03 +0200 If you configure a biarch Linux/Sparc compiler defaulting to 32-bit, but give --with-cpu= for a v9 cpu it erroneously turns on 64-bit in TARGET_DEFAULT. PR target/50354 reports the breakage of the opposite case

Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread Eric Botcazou
If one wants a 32-bit default compiler, they should build for the sparc-linux target. And this is absolutely trivial to make happen in the environments where this is supposedly a problem. I have criticized so many times this combination in the past, while Jakub and also you IIRC were

Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread David Miller
From: Eric Botcazou ebotca...@adacore.com Date: Fri, 14 Oct 2011 22:31:53 +0200 If one wants a 32-bit default compiler, they should build for the sparc-linux target. And this is absolutely trivial to make happen in the environments where this is supposedly a problem. I have criticized so

Re: [PATCH] Fix target default on biarch Linux/Sparc

2011-10-14 Thread Eric Botcazou
Agreed, please install this patch if you haven't already. Thanks, done after successfully bootstrapping on SPARC64/Linux. -- Eric Botcazou

[PATCH] Fix target default on biarch Linux/Sparc

2011-09-05 Thread David Miller
If you configure a biarch Linux/Sparc compiler defaulting to 32-bit, but give --with-cpu= for a v9 cpu it erroneously turns on 64-bit in TARGET_DEFAULT. The right thing to do is what the Solaris/Sparc target does, which is to key things off of a cpp macro (TARGET_64BIT_DEFAULT) which is defined