Re: [PATCH] Handle VEC_SERIES with both constant args in simplify_binary_operation

[Richard Sandiford <richard.sandiford@linaro.org>]

Jakub Jelinek <jakub@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  <jakub@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