[Bug target/84422] ICE on various builtin test functions when compiled with -mcpu=power7

[carll at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84422

Carl Love  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #6 from Carl Love  ---
gcc/testsuite/gcc.target/powerpc/builtin-fctid-fctiw-runnable.c
 was fixed by reverting the patch that added them.  Commit 258492

gcc/testsuite/gcc.target/powerpc/sse2-pmuludq-1.c
  was fixed by commit 258539

At this point all of the test cases that were generating and ICE for
-mcpu=power7 have been fixed.  


Closing the issue.

[Bug target/84422] ICE on various builtin test functions when compiled with -mcpu=power7

[carll at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84422

--- Comment #5 from Carl Love  ---
Author: carll
Date: Wed Mar 14 23:01:12 2018
New Revision: 258539

URL: https://gcc.gnu.org/viewcvs?rev=258539=gcc=rev
Log:
gcc/ChangeLog:

2018-03-14  Carl Love  

PR target/84422
* config/rs6000/rs6000-builtin.def: Change expansion for
VMULESW to BU_P8V_AV_2.
Change expansion for VMULEUW to BU_P8V_AV_2.
* config/rs6000/rs6000.c: Change
ALTIVEC_BUILTIN_VMULESW to P8V_BUILTIN_VMULESW.
Change ALTIVEC_BUILTIN_VMULEUW to P8V_BUILTIN_VMULEUW.
Change ALTIVEC_BUILTIN_VMULOSW to P8V_BUILTIN_VMULOSW.
Change ALTIVEC_BUILTIN_VMULOUW to P8V_BUILTIN_VMULOUW.
* config/rs6000/rs6000-c.c: Change
ALTIVEC_BUILTIN_VMULESW to P8V_BUILTIN_VMULESW.
Change ALTIVEC_BUILTIN_VMULEUW to P8V_BUILTIN_VMULEUW.
Change ALTIVEC_BUILTIN_VMULOSW to P8V_BUILTIN_VMULOSW.
Change ALTIVEC_BUILTIN_VMULOUW to P8V_BUILTIN_VMULOUW.

Modified:
trunk/gcc/config/rs6000/rs6000-builtin.def
trunk/gcc/config/rs6000/rs6000-c.c
trunk/gcc/config/rs6000/rs6000.c

[Bug target/84422] ICE on various builtin test functions when compiled with -mcpu=power7

[carll at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84422

--- Comment #4 from Carl Love  ---
gcc/testsuite/gcc.target/powerpc/fold-vec-neg-longlong.p8.c
during RTL pass: vregs

gcc/testsuite/gcc.target/powerpc/fold-vec-neg-longlong.p9.c
during RTL pass: vregs

Both tests fixed with mainline commit 258006 on 2/26/2018

[Bug target/84422] ICE on various builtin test functions when compiled with -mcpu=power7

[carll at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84422

--- Comment #3 from Carl Love  ---
gcc/testsuite/gcc.target/powerpc/sse2-pmuludq-1.c

Test requres Power 8 as a minimum.  Compiling with -mcpu=power7 generates an
ICE but the test requires Power8.  So the ICE shouldn't be an issue under
normal testing conditions.

[Bug target/84422] ICE on various builtin test functions when compiled with -mcpu=power7

[carll at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84422

--- Comment #2 from Carl Love  ---
Moved Power 8 vec_float2, vec_signed2 and vec_unsigned2 builtin tests to new
file builtins-3-runnable-p8.c.  Fixed ICE for vec_signed2 and vec_unsigned2
which were found in builtins-3-runnable.c once the vec_float2 test was moved to
a P8 test file.  Commit 257937.

[Bug target/84422] ICE on various builtin test functions when compiled with -mcpu=power7

[carll at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84422

Carl Love  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2018-02-16
   Assignee|unassigned at gcc dot gnu.org  |carll at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #1 from Carl Love  ---
GCC trunk commit 257752 fixed the issue with:

  gcc/testsuite/gcc.target/powerpc/builtins-3-runnable.c
   during RTL pass: vregs