This is the mail archive of the
gcc-prs@gcc.gnu.org
mailing list for the GCC project.
target/7693: Typo in i386 mmintrin.h header
- From: shawnw at speakeasy dot org
- To: gcc-gnats at gcc dot gnu dot org
- Date: Fri, 23 Aug 2002 00:33:53 -0700
- Subject: target/7693: Typo in i386 mmintrin.h header
>Number: 7693
>Category: target
>Synopsis: Typo in i386 mmintrin.h header
>Confidential: no
>Severity: non-critical
>Priority: medium
>Responsible: unassigned
>State: open
>Class: sw-bug
>Submitter-Id: net
>Arrival-Date: Fri Aug 23 00:36:01 PDT 2002
>Closed-Date:
>Last-Modified:
>Originator: Shawn Wagner
>Release: 3.2
>Organization:
>Environment:
System: Linux sherlock.localdomain 2.2.21 #1 Sun Aug 18 11:20:19 PDT 2002 i686 unknown
Architecture: athlon
host: athlon-pc-linux-gnu
build: athlon-pc-linux-gnu
target: athlon-pc-linux-gnu
configured with: ../gcc-3.2/configure --prefix=/usr/local --target=athlon-pc-linux-gnu --with-arch=athlon --enable-threads=posix --disable-nls --with-system-zlib --enable-languages=c,c++,f77 --enable-__cxa_atexit --enable-shared
>Description:
The functions in gcc/config/i386/mmintrin.h for shifting 64 bit values don't
match the names in the Intel x86 instruction set documentation, or what the
Intel compiler accepts. In all 4 relevant functions, the pi64 extension in
the gcc mmintrin.h names should be si64. For example, _mm_sll_pi64() should
be _mm_sll_si64().
Problem is in 3.2 and current cvs HEAD.
>How-To-Repeat:
>Fix:
--- gcc/config/i386/mmintrin.h.orig Fri Aug 23 00:06:52 2002
+++ gcc/config/i386/mmintrin.h Fri Aug 23 00:10:40 2002
@@ -293,13 +293,13 @@
/* Shift the 64-bit value in M left by COUNT. */
static __inline __m64
-_mm_sll_pi64 (__m64 __m, __m64 __count)
+_mm_sll_si64 (__m64 __m, __m64 __count)
{
return (__m64) __builtin_ia32_psllq (__m, __count);
}
static __inline __m64
-_mm_slli_pi64 (__m64 __m, int __count)
+_mm_slli_si64 (__m64 __m, int __count)
{
return (__m64) __builtin_ia32_psllq (__m, __count);
}
@@ -358,13 +358,13 @@
/* Shift the 64-bit value in M left by COUNT; shift in zeros. */
static __inline __m64
-_mm_srl_pi64 (__m64 __m, __m64 __count)
+_mm_srl_si64 (__m64 __m, __m64 __count)
{
return (__m64) __builtin_ia32_psrlq (__m, __count);
}
static __inline __m64
-_mm_srli_pi64 (__m64 __m, int __count)
+_mm_srli_si64 (__m64 __m, int __count)
{
return (__m64) __builtin_ia32_psrlq (__m, __count);
}
>Release-Note:
>Audit-Trail:
>Unformatted: