On Tue, Aug 25, 2015 at 10:14 AM, Pat Haugen
<pthau...@linux.vnet.ibm.com> wrote:
> The following patch fixes the vec_shr expander to do a shift instead of a
> rotate. CPU2006 benchmark 482.sphinx3 recently started failing due to this
> issue.  Bootstrapped and tested on ppc64/ppc64le with no new regressions. Ok
> for trunk? And ok for 4.9/5 (with equivalent change to vec_shl expander
> which exists in those releases) after bootstrap/regtest?
>
> -Pat
>
>
> 2015-08-25  Pat Haugen  <pthau...@us.ibm.com>
>
>         * config/rs6000/vector.md (vec_shr_<mode>): Fix to do a shift
>         instead of a rotate.
>
> gcc/testsuite:
>         * gcc.target/powerpc/vec-shr.c: New.

This is okay.

As Peter and I noticed

+  zero_reg = gen_reg_rtx(<MODE>mode);

This needs a space after gen_rtx_rtx.

Thanks, David

Reply via email to