Disable some VEC_COND_EXPR transformations after vector lowering
authorMarc Glisse <marc.glisse@inria.fr>
Fri, 7 Aug 2020 16:49:04 +0000 (18:49 +0200)
committerMarc Glisse <marc.glisse@inria.fr>
Fri, 7 Aug 2020 16:49:04 +0000 (18:49 +0200)
commita1ee6d507b0c26466be519d177f5a08b22f63647
tree5da578f99b7f1a468969a38129f9f4a345353e05
parent6c3ae88d1e13b71665d1b27821159dcbea410267
Disable some VEC_COND_EXPR transformations after vector lowering

ARM understands VEC_COND_EXPR<v == w, -1, 0> but not a plain v == w which is
fed to something other than VEC_COND_EXPR (say BIT_IOR_EXPR). This patch avoids
introducing the second kind of statement after the vector lowering pass, which
is the last chance to turn v == w back into something the target handles.

This is just a workaround to avoid ICEs, a v == w produced before vector
lowering will yield pretty bad code. Either the arm target needs to learn to
handle vector comparisons (aarch64 already does), or the middle-end needs to
fall back to vcond when plain comparisons are not supported (or ...).

2020-08-07  Marc Glisse  <marc.glisse@inria.fr>

* generic-match-head.c (optimize_vectors_before_lowering_p): New
function.
* gimple-match-head.c (optimize_vectors_before_lowering_p):
Likewise.
* match.pd ((v ? w : 0) ? a : b, c1 ? c2 ? a : b : b): Use it.
gcc/generic-match-head.c
gcc/gimple-match-head.c
gcc/match.pd