[Bug target/63600] [5 Regression] ice in ix86_expand_sse2_abs

2014-10-23 Thread kyukhin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63600

--- Comment #5 from Kirill Yukhin kyukhin at gcc dot gnu.org ---
Created attachment 33793
  -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=33793action=edit
Updated patch + regtest

Will check in after testing is over

[Bug target/63600] [5 Regression] ice in ix86_expand_sse2_abs

2014-10-23 Thread kyukhin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63600

--- Comment #6 from Kirill Yukhin kyukhin at gcc dot gnu.org ---
(In reply to Kirill Yukhin from comment #5)
 Created attachment 33793 [details]
 Updated patch + regtest
 
 Will check in after testing is over

Checked into main trunk (forgot to put PR into ChangeLogs).
https://gcc.gnu.org/ml/gcc-cvs/2014-10/msg00853.html

[Bug target/63600] [5 Regression] ice in ix86_expand_sse2_abs

2014-10-23 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63600

Jakub Jelinek jakub at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #7 from Jakub Jelinek jakub at gcc dot gnu.org ---
Author: kyukhin
Date: Thu Oct 23 11:50:19 2014
New Revision: 216591

URL: https://gcc.gnu.org/viewcvs?rev=216591root=gccview=rev
Log:
AVX. Fix block absq emit for non AVX-512 targets.

gcc/
* config/i386/sse.md (define_mode_iterator VI1248_AVX512VL_AVX512BW):
New.
(define_insn *absmode2): Use VI1248_AVX512VL_AVX512BW mode
iterator.
(define_expand absmode2): Ditto.

gcc/testsuite/
* gcc.target/i386/pr63600.c: New.

Added:
trunk/gcc/testsuite/gcc.target/i386/pr63600.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/sse.md
trunk/gcc/testsuite/ChangeLog

[Bug target/63600] [5 Regression] ice in ix86_expand_sse2_abs

2014-10-22 Thread kyukhin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63600

--- Comment #2 from Kirill Yukhin kyukhin at gcc dot gnu.org ---
Created attachment 33779
  -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=33779action=edit
Patch is under testing now.

[Bug target/63600] [5 Regression] ice in ix86_expand_sse2_abs

2014-10-22 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63600

--- Comment #3 from Jakub Jelinek jakub at gcc dot gnu.org ---
I would have expected VI instead of IV in the iterator name, but I never
understood the rules for i?86 iterator names.
And, you want the testcase in the testsuite too.

[Bug target/63600] [5 Regression] ice in ix86_expand_sse2_abs

2014-10-22 Thread kyukhin at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63600

--- Comment #4 from Kirill Yukhin kyukhin at gcc dot gnu.org ---
(In reply to Jakub Jelinek from comment #3)
 I would have expected VI instead of IV in the iterator name, but I never
 understood the rules for i?86 iterator names.
 And, you want the testcase in the testsuite too.

Agh, right! Thanks. I'll post patch w/ renamed iterator and
a testcase (from this bug)

[Bug target/63600] [5 Regression] ice in ix86_expand_sse2_abs

2014-10-20 Thread jakub at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63600

Jakub Jelinek jakub at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2014-10-20
 CC||jakub at gcc dot gnu.org
   Target Milestone|--- |5.0
Summary|ice in ix86_expand_sse2_abs |[5 Regression] ice in
   ||ix86_expand_sse2_abs
 Ever confirmed|0   |1

--- Comment #1 from Jakub Jelinek jakub at gcc dot gnu.org ---
Started with r216255.  Reduced testcase for -O3:
long *a, b;
int c;
void
foo (void)
{
  for (c = 0; c  64; c++)
a[c] = b = 0 ? b : -b;
}