powerpc: Fix ICE with fp conditional move (PR target/93073)
authorJakub Jelinek <jakub@redhat.com>
Tue, 21 Jan 2020 08:17:27 +0000 (09:17 +0100)
committerJakub Jelinek <jakub@redhat.com>
Tue, 21 Jan 2020 08:17:27 +0000 (09:17 +0100)
commit51e010b5f75c1fff06425a72702c1bf82a3ab053
treef38a85230ecad134b073df790ac3f24ea6b8a7dd
parent7d593fd672e5a01610d750c754213ea6c3e71ef1
powerpc: Fix ICE with fp conditional move (PR target/93073)

The following testcase ICEs, because for TFmode the particular subtraction
pattern (*subtf3) is not enabled with the given options.  Using
expand_simple_binop instead of emitting the subtraction by hand just moves
the ICE one insn later, NEG of ABS is not then recognized, etc., but
ultimately the problem is that when rs6000_emit_cmove is called for floating
point operand mode (and earlier condition ensures that in that case
compare_mode is also floating point), the expander makes sure the
operand mode is SFDF, but for the comparison mode nothing checks it, yet
there is just one *fsel* pattern with 2 separate SFDF iterators.

The following patch fixes it by giving up if compare_mode is not SFmode or
DFmode.

2020-01-21  Jakub Jelinek  <jakub@redhat.com>

PR target/93073
* config/rs6000/rs6000.c (rs6000_emit_cmove): If using fsel, punt for
compare_mode other than SFmode or DFmode.

* gcc.target/powerpc/pr93073.c: New test.
gcc/ChangeLog
gcc/config/rs6000/rs6000.c
gcc/testsuite/ChangeLog
gcc/testsuite/gcc.target/powerpc/pr93073.c [new file with mode: 0644]