Patch to fix build breakage with GCC 4.7

[David Holmes]

On 31/01/2012 12:36 PM, Deepak Bhole wrote:
> * David Holmes<david.holmes at oracle.com>  [2012-01-30 19:57]:
>> While gcc compilation on sparc is rare I'm not sure that simply
>> deleting the sparc-only option unconditionally is the right thing to
>> do.
>>
>
> I thought about that too. But I was unable to find info on OpenJDK +
> SPARC + Linux. Is that combination even supported? The README doesn't
> list it:
> http://hg.openjdk.java.net/jdk6/jdk6/raw-file/tip/README-builds.html#MBE

Linux-sparc is not one of Oracle's supported OpenJDK platforms. However 
AFAIK there are people in the community building OpenJDK on Linux-sparc 
using the Zero interpreter. I don't know if this would affect them but 
it still seems to me that we should be careful not to break other 
people's builds.

David
-----

> The option seemed more like a relic from Solaris + SPARC config rather
> than a requirement for Linux + SPARC.



> Cheers,
> Deepak
>
>> David
>>
>> On 31/01/2012 1:20 AM, Deepak Bhole wrote:
>>> Hi,
>>>
>>> JDK builds currently fail with GCC 4.7 due to its stricter option
>>> checking.
>>>
>>> GCC 4.6 and prior ignored invalid options -- GCC 4.7 does not. Certain
>>> files in JDK supply the -mimpure-text option to GCC. This option is only
>>> valid on SPARC[1,2]. As a result, GCC 4.7 throws an error during build
>>> on Linux (I suppose .
>>>
>>> This patch removes the option:
>>> http://cr.openjdk.java.net/~dbhole/GCC-4.7-JDK8.00
>>>
>>> 1: http://gcc.gnu.org/onlinedocs/gcc-3.3.6/gcc/SPARC-Options.html
>>> 2: http://gcc.gnu.org/onlinedocs/gcc-3.3.6/gcc/i386-and-x86_002d64-Options.html
>>>
>>> If OK for push, please feel free to do so (I don't have commit access).
>>>
>>> Cheers,
>>> Deepak