Patchwork Fix assembler options for -mcpu={supersparc,hypersparc}

login
register
mail settings
Submitter David Miller
Date April 10, 2013, 7:19 p.m.
Message ID <20130410.151957.1054425473718671658.davem@davemloft.net>
Download mbox | patch
Permalink /patch/235452/
State New
Headers show

Comments

David Miller - April 10, 2013, 7:19 p.m.
This is yet another bug just like PR target/52610, we need to pass
-Av8 to the assembler for every cpu type that will use MASK_V8 in
sparc.c:sparc_option_override().

I found this while testing GMP builds.

I'd like to check this into the various gcc-4_X-branch branches as
well, unless there are major objections.

gcc/

	* config/sparc/sparc.h (ASM_CPU_SPEC): Pass -Av8 if -mcpu=supersparc
	or -mcpu=hypersparc.
Eric Botcazou - April 11, 2013, 7:42 a.m.
> This is yet another bug just like PR target/52610, we need to pass
> -Av8 to the assembler for every cpu type that will use MASK_V8 in
> sparc.c:sparc_option_override().

What on Earth are you doing with -mcpu=supersparc and -mcpu=hypersparc? :-)

> I found this while testing GMP builds.
> 
> I'd like to check this into the various gcc-4_X-branch branches as
> well, unless there are major objections.

Sure, PR target/52610 was fixed for 4.7.x and above so you could do the same.

Patch

diff --git a/gcc/config/sparc/sparc.h b/gcc/config/sparc/sparc.h
index 6b02b45..c6122c1 100644
--- a/gcc/config/sparc/sparc.h
+++ b/gcc/config/sparc/sparc.h
@@ -327,6 +327,8 @@  extern enum cmodel sparc_cmodel;
 %{mcpu=sparclite86x:-Asparclite} \
 %{mcpu=f930:-Asparclite} %{mcpu=f934:-Asparclite} \
 %{mcpu=v8:-Av8} \
+%{mcpu=supersparc:-Av8} \
+%{mcpu=hypersparc:-Av8} \
 %{mcpu=leon:-Av8} \
 %{mv8plus:-Av8plus} \
 %{mcpu=v9:-Av9} \