This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
ARM: correct some builtin names
- From: Ben Elliston <bje at wasabisystems dot com>
- To: gcc-patches at gcc dot gnu dot org
- Date: 09 Jan 2004 08:34:17 +1100
- Subject: ARM: correct some builtin names
I noticed that two builtins were incorrectly named after reading the
iWMMXt developers guide. I suspect the original author somehow
misread "M" as "H". Okay to commit?
Ben
2004-01-08 Ben Elliston <bje@wasabisystems.com>
* config/arm/arm.c (bdesc_2arg): Correct builtin names "wmulsh"
and "wmuluh" to "wmulsm" and "wmulum", respectively.
* config/arm/arm.h (enum arm_builtins): Rename enumerators to
ARM_BUILTIN_WMULSM and ARM_BUILTIN_WMULUM.
* config/arm/mmintrin.h (_mm_mulhi_pi16): Update intrinsic call.
(_mm_mulhi_pu16): Likewise.
Index: config/arm/arm.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/arm/arm.c,v
retrieving revision 1.316
diff -u -p -r1.316 arm.c
--- config/arm/arm.c 25 Dec 2003 15:17:36 -0000 1.316
+++ config/arm/arm.c 8 Jan 2004 21:28:30 -0000
@@ -10608,8 +10608,8 @@ static const struct builtin_description
IWMMXT_BUILTIN (ussubv4hi3, "wsubhus", WSUBUSH)
IWMMXT_BUILTIN (ussubv2si3, "wsubwus", WSUBUSW)
IWMMXT_BUILTIN (mulv4hi3, "wmulul", WMULUL)
- IWMMXT_BUILTIN (smulv4hi3_highpart, "wmulsh", WMULSH)
- IWMMXT_BUILTIN (umulv4hi3_highpart, "wmuluh", WMULUH)
+ IWMMXT_BUILTIN (smulv4hi3_highpart, "wmulsm", WMULSM)
+ IWMMXT_BUILTIN (umulv4hi3_highpart, "wmulum", WMULUM)
IWMMXT_BUILTIN (eqv8qi3, "wcmpeqb", WCMPEQB)
IWMMXT_BUILTIN (eqv4hi3, "wcmpeqh", WCMPEQH)
IWMMXT_BUILTIN (eqv2si3, "wcmpeqw", WCMPEQW)
Index: config/arm/arm.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/arm/arm.h,v
retrieving revision 1.213
diff -u -p -r1.213 arm.h
--- config/arm/arm.h 25 Dec 2003 15:17:36 -0000 1.213
+++ config/arm/arm.h 8 Jan 2004 21:28:35 -0000
@@ -2857,8 +2857,8 @@ enum arm_builtins
ARM_BUILTIN_WMINUH,
ARM_BUILTIN_WMINUB,
- ARM_BUILTIN_WMULUH,
- ARM_BUILTIN_WMULSH,
+ ARM_BUILTIN_WMULUM,
+ ARM_BUILTIN_WMULSM,
ARM_BUILTIN_WMULUL,
ARM_BUILTIN_PSADBH,
Index: config/arm/mmintrin.h
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/arm/mmintrin.h,v
retrieving revision 1.1
diff -u -p -r1.1 mmintrin.h
--- config/arm/mmintrin.h 18 Jun 2003 16:36:13 -0000 1.1
+++ config/arm/mmintrin.h 8 Jan 2004 21:28:37 -0000
@@ -419,7 +419,7 @@ _mm_madd_pu16 (__m64 __m1, __m64 __m2)
static __inline __m64
_mm_mulhi_pi16 (__m64 __m1, __m64 __m2)
{
- return (__m64) __builtin_arm_wmulsh ((__v4hi)__m1, (__v4hi)__m2);
+ return (__m64) __builtin_arm_wmulsm ((__v4hi)__m1, (__v4hi)__m2);
}
/* Multiply four signed 16-bit values in M1 by four signed 16-bit values in
@@ -427,7 +427,7 @@ _mm_mulhi_pi16 (__m64 __m1, __m64 __m2)
static __inline __m64
_mm_mulhi_pu16 (__m64 __m1, __m64 __m2)
{
- return (__m64) __builtin_arm_wmuluh ((__v4hi)__m1, (__v4hi)__m2);
+ return (__m64) __builtin_arm_wmulum ((__v4hi)__m1, (__v4hi)__m2);
}
/* Multiply four 16-bit values in M1 by four 16-bit values in M2 and produce