Jakub Jelinek <ja...@redhat.com> writes: > Hi! > > I've noticed that if we construct a VEC_SERIES and later propagate > constants into both arguments (could happen e.g. in DEBUG_INSNs, or > during combine etc.) we don't simplify that. > > The following patch adds that, ok for trunk if it passes bootstrap/regtest? > > Or should it go into simplify_constant_binary_operation instead? > In there it would better match the function name, on the other side > it would slow down the function for quite a marginal case, because it > doesn't have a big switch on code, but instead handles just a few cases > if constant operand kinds vs. the mode. > > 2017-11-22 Jakub Jelinek <ja...@redhat.com> > > * simplify-rtx.c (simplify_binary_operation_1) <case VEC_SERIES>: > Handle the case where both arguments are using gen_const_vec_series.
LGTM FWIW. Thanks for doing this. Richard
