Re: target/7693: Typo in i386 mmintrin.h header

Re: target/7693: Typo in i386 mmintrin.h header

[hubicka]

Synopsis: Typo in i386 mmintrin.h header

Responsible-Changed-From-To: unassigned->bernds
Responsible-Changed-By: hubicka
Responsible-Changed-When: Thu Oct  3 10:39:07 2002
Responsible-Changed-Why:
    Bernd, looks like trivial for you to approve this fix

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=7693

Re: target/7693: Typo in i386 mmintrin.h header

[hubicka]

Synopsis: Typo in i386 mmintrin.h header

State-Changed-From-To: open->closed
State-Changed-By: hubicka
State-Changed-When: Sun Oct 27 02:24:46 2002
State-Changed-Why:
    Fixed.

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=7693

target/7693: Typo in i386 mmintrin.h header

[shawnw]

>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: