diff mbox series

m68k: Use M68K_SCALE_AVAILABLE on __mpn_lshift and __mpn_rshift

Message ID 20230818183559.2081164-1-adhemerval.zanella@linaro.org
State New
Headers show
Series m68k: Use M68K_SCALE_AVAILABLE on __mpn_lshift and __mpn_rshift | expand

Commit Message

Adhemerval Zanella Aug. 18, 2023, 6:35 p.m. UTC 
This patch adds a new macro, M68K_SCALE_AVAILABLE, similar to gmp
scale_available_p (mpn/m68k/m68k-defs.m4) that expand to 1 if a
scale factor can be used in addressing modes.  This is used
instead of __mc68020__ for some optimization decisions.

Checked on a build for m68k-linux-gnu target mc68020 and mc68040.
---
 sysdeps/m68k/m680x0/lshift.S | 6 +++---
 sysdeps/m68k/m680x0/rshift.S | 8 ++++----
 sysdeps/m68k/sysdep.h        | 7 +++++++
 3 files changed, 14 insertions(+), 7 deletions(-)

Comments

Andreas Schwab Aug. 18, 2023, 7:05 p.m. UTC | #1 
On Aug 18 2023, Adhemerval Zanella wrote:

> diff --git a/sysdeps/m68k/sysdep.h b/sysdeps/m68k/sysdep.h
> index dfc8aebc3a..a1e89b87d5 100644
> --- a/sysdeps/m68k/sysdep.h
> +++ b/sysdeps/m68k/sysdep.h
> @@ -72,3 +72,10 @@
>  # endif
>  
>  #endif	/* __ASSEMBLER__ */
> +
> +# if defined __mc68020__ || defined __mc68030__ || defined __mc68040__	      \
> +     || defined __mc68060__
> +#  define M68K_SCALE_AVAILABLE 1
> +# else
> +#  define M68K_SCALE_AVAILABLE 0
> +# endif

Since this is only for sources in m68k/m680x0, and only handles classic
m68k, it should be put in sysdeps/m68k/m680x0/sysdep.h

Ok with that change.
diff mbox series

Patch

diff --git a/sysdeps/m68k/m680x0/lshift.S b/sysdeps/m68k/m680x0/lshift.S
index 4240738959..74de90380b 100644
--- a/sysdeps/m68k/m680x0/lshift.S
+++ b/sysdeps/m68k/m680x0/lshift.S
@@ -57,9 +57,9 @@  ENTRY(__mpn_lshift)
 	bne	L(Lnormal)
 	cmpl	R(s_ptr),R(res_ptr)
 	bls	L(Lspecial)		/* jump if s_ptr >= res_ptr */
-#if (defined (__mc68020__) || defined (__NeXT__) || defined(mc68020))
+#if M68K_SCALE_AVAILABLE
 	lea	MEM_INDX1(s_ptr,s_size,l,4),R(a2)
-#else /* not mc68020 */
+#else
 	movel	R(s_size),R(d0)
 	asll	#2,R(d0)
 	lea	MEM_INDX(s_ptr,d0,l),R(a2)
@@ -71,7 +71,7 @@  L(Lnormal:)
 	moveql	#32,R(d5)
 	subl	R(cnt),R(d5)
 
-#if (defined (__mc68020__) || defined (__NeXT__) || defined(mc68020))
+#if M68K_SCALE_AVAILABLE
 	lea	MEM_INDX1(s_ptr,s_size,l,4),R(s_ptr)
 	lea	MEM_INDX1(res_ptr,s_size,l,4),R(res_ptr)
 #else /* not mc68000 */
diff --git a/sysdeps/m68k/m680x0/rshift.S b/sysdeps/m68k/m680x0/rshift.S
index d56a2b4a44..d398d0287e 100644
--- a/sysdeps/m68k/m680x0/rshift.S
+++ b/sysdeps/m68k/m680x0/rshift.S
@@ -56,9 +56,9 @@  ENTRY(__mpn_rshift)
 	bne	L(Lnormal)
 	cmpl	R(res_ptr),R(s_ptr)
 	bls	L(Lspecial)		/* jump if res_ptr >= s_ptr */
-#if (defined (__mc68020__) || defined (__NeXT__) || defined(mc68020))
+#if M68K_SCALE_AVAILABLE
 	lea	MEM_INDX1(res_ptr,s_size,l,4),R(a2)
-#else /* not mc68020 */
+#else
 	movel	R(s_size),R(d0)
 	asll	#2,R(d0)
 	lea	MEM_INDX(res_ptr,d0,l),R(a2)
@@ -121,10 +121,10 @@  L(Lend:)
 
 	cfi_restore_state
 L(Lspecial:)
-#if (defined (__mc68020__) || defined (__NeXT__) || defined(mc68020))
+#if M68K_SCALE_AVAILABLE
 	lea	MEM_INDX1(s_ptr,s_size,l,4),R(s_ptr)
 	lea	MEM_INDX1(res_ptr,s_size,l,4),R(res_ptr)
-#else /* not mc68000 */
+#else
 	movel	R(s_size),R(d0)
 	asll	#2,R(d0)
 	addl	R(d0),R(s_ptr)
diff --git a/sysdeps/m68k/sysdep.h b/sysdeps/m68k/sysdep.h
index dfc8aebc3a..a1e89b87d5 100644
--- a/sysdeps/m68k/sysdep.h
+++ b/sysdeps/m68k/sysdep.h
@@ -72,3 +72,10 @@ 
 # endif
 
 #endif	/* __ASSEMBLER__ */
+
+# if defined __mc68020__ || defined __mc68030__ || defined __mc68040__	      \
+     || defined __mc68060__
+#  define M68K_SCALE_AVAILABLE 1
+# else
+#  define M68K_SCALE_AVAILABLE 0
+# endif