public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/47840] New: incorrect _mm256_insert_epi{32,64} implementations
@ 2011-02-21 18:51 kretz at kde dot org
2011-02-21 20:00 ` [Bug target/47840] [4.4/4.5/4.6 Regression] " ubizjak at gmail dot com
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: kretz at kde dot org @ 2011-02-21 18:51 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47840
Summary: incorrect _mm256_insert_epi{32,64} implementations
Product: gcc
Version: 4.6.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: target
AssignedTo: unassigned@gcc.gnu.org
ReportedBy: kretz@kde.org
see avxintrin.h:
_mm256_insert_epi32 and _mm256_insert_epi64 use _mm_insert_epi16 internally
if __OPTIMIZE__ is not defined they use _mm_insert_epi32/64 internally - as I
would have expected.
I also checked 4.5 and 4.4, and it seems the bug was not noticed in any branch
yet.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/47840] [4.4/4.5/4.6 Regression] incorrect _mm256_insert_epi{32,64} implementations
2011-02-21 18:51 [Bug target/47840] New: incorrect _mm256_insert_epi{32,64} implementations kretz at kde dot org
@ 2011-02-21 20:00 ` ubizjak at gmail dot com
2011-02-21 20:05 ` uros at gcc dot gnu.org
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: ubizjak at gmail dot com @ 2011-02-21 20:00 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47840
Uros Bizjak <ubizjak at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed| |2011.02.21 19:45:17
AssignedTo|unassigned at gcc dot |ubizjak at gmail dot com
|gnu.org |
Target Milestone|--- |4.4.6
Summary|incorrect |[4.4/4.5/4.6 Regression]
|_mm256_insert_epi{32,64} |incorrect
|implementations |_mm256_insert_epi{32,64}
| |implementations
Ever Confirmed|0 |1
--- Comment #1 from Uros Bizjak <ubizjak at gmail dot com> 2011-02-21 19:45:17 UTC ---
A typo, following patch should fix it:
Index: avxintrin.h
===================================================================
--- avxintrin.h (revision 170367)
+++ avxintrin.h (working copy)
@@ -737,7 +737,7 @@
_mm256_insert_epi32 (__m256i __X, int __D, int const __N)
{
__m128i __Y = _mm256_extractf128_si256 (__X, __N >> 2);
- __Y = _mm_insert_epi16 (__Y, __D, __N % 4);
+ __Y = _mm_insert_epi32 (__Y, __D, __N % 4);
return _mm256_insertf128_si256 (__X, __Y, __N >> 2);
}
@@ -762,7 +762,7 @@
_mm256_insert_epi64 (__m256i __X, int __D, int const __N)
{
__m128i __Y = _mm256_extractf128_si256 (__X, __N >> 1);
- __Y = _mm_insert_epi16 (__Y, __D, __N % 2);
+ __Y = _mm_insert_epi64 (__Y, __D, __N % 2);
return _mm256_insertf128_si256 (__X, __Y, __N >> 1);
}
#endif
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/47840] [4.4/4.5/4.6 Regression] incorrect _mm256_insert_epi{32,64} implementations
2011-02-21 18:51 [Bug target/47840] New: incorrect _mm256_insert_epi{32,64} implementations kretz at kde dot org
2011-02-21 20:00 ` [Bug target/47840] [4.4/4.5/4.6 Regression] " ubizjak at gmail dot com
@ 2011-02-21 20:05 ` uros at gcc dot gnu.org
2011-02-21 20:06 ` uros at gcc dot gnu.org
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: uros at gcc dot gnu.org @ 2011-02-21 20:05 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47840
--- Comment #2 from uros at gcc dot gnu.org 2011-02-21 19:59:55 UTC ---
Author: uros
Date: Mon Feb 21 19:59:52 2011
New Revision: 170371
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170371
Log:
PR target/47840
* config/i386/avxintrin.h (_mm256_insert_epi32): Use _mm_insert_epi32.
(_mm256_insert_epi64): Use _mm_insert_epi64.
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/avxintrin.h
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/47840] [4.4/4.5/4.6 Regression] incorrect _mm256_insert_epi{32,64} implementations
2011-02-21 18:51 [Bug target/47840] New: incorrect _mm256_insert_epi{32,64} implementations kretz at kde dot org
2011-02-21 20:00 ` [Bug target/47840] [4.4/4.5/4.6 Regression] " ubizjak at gmail dot com
2011-02-21 20:05 ` uros at gcc dot gnu.org
@ 2011-02-21 20:06 ` uros at gcc dot gnu.org
2011-02-21 20:59 ` uros at gcc dot gnu.org
2011-02-21 21:02 ` ubizjak at gmail dot com
4 siblings, 0 replies; 6+ messages in thread
From: uros at gcc dot gnu.org @ 2011-02-21 20:06 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47840
--- Comment #3 from uros at gcc dot gnu.org 2011-02-21 20:02:08 UTC ---
Author: uros
Date: Mon Feb 21 20:02:02 2011
New Revision: 170372
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170372
Log:
PR target/47840
* config/i386/avxintrin.h (_mm256_insert_epi32): Use _mm_insert_epi32.
(_mm256_insert_epi64): Use _mm_insert_epi64.
Modified:
branches/gcc-4_5-branch/gcc/ChangeLog
branches/gcc-4_5-branch/gcc/config/i386/avxintrin.h
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/47840] [4.4/4.5/4.6 Regression] incorrect _mm256_insert_epi{32,64} implementations
2011-02-21 18:51 [Bug target/47840] New: incorrect _mm256_insert_epi{32,64} implementations kretz at kde dot org
` (2 preceding siblings ...)
2011-02-21 20:06 ` uros at gcc dot gnu.org
@ 2011-02-21 20:59 ` uros at gcc dot gnu.org
2011-02-21 21:02 ` ubizjak at gmail dot com
4 siblings, 0 replies; 6+ messages in thread
From: uros at gcc dot gnu.org @ 2011-02-21 20:59 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47840
--- Comment #4 from uros at gcc dot gnu.org 2011-02-21 20:05:02 UTC ---
Author: uros
Date: Mon Feb 21 20:04:57 2011
New Revision: 170373
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170373
Log:
PR target/47840
* config/i386/avxintrin.h (_mm256_insert_epi32): Use _mm_insert_epi32.
(_mm256_insert_epi64): Use _mm_insert_epi64.
Modified:
branches/gcc-4_4-branch/gcc/ChangeLog
branches/gcc-4_4-branch/gcc/config/i386/avxintrin.h
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug target/47840] [4.4/4.5/4.6 Regression] incorrect _mm256_insert_epi{32,64} implementations
2011-02-21 18:51 [Bug target/47840] New: incorrect _mm256_insert_epi{32,64} implementations kretz at kde dot org
` (3 preceding siblings ...)
2011-02-21 20:59 ` uros at gcc dot gnu.org
@ 2011-02-21 21:02 ` ubizjak at gmail dot com
4 siblings, 0 replies; 6+ messages in thread
From: ubizjak at gmail dot com @ 2011-02-21 21:02 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47840
Uros Bizjak <ubizjak at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
--- Comment #5 from Uros Bizjak <ubizjak at gmail dot com> 2011-02-21 20:06:03 UTC ---
Fixed.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-02-21 20:06 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-21 18:51 [Bug target/47840] New: incorrect _mm256_insert_epi{32,64} implementations kretz at kde dot org
2011-02-21 20:00 ` [Bug target/47840] [4.4/4.5/4.6 Regression] " ubizjak at gmail dot com
2011-02-21 20:05 ` uros at gcc dot gnu.org
2011-02-21 20:06 ` uros at gcc dot gnu.org
2011-02-21 20:59 ` uros at gcc dot gnu.org
2011-02-21 21:02 ` ubizjak at gmail dot com
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).