This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 02/13] optabs: Fix vec_perm -> V16QI middle end lowering.
- From: Andreas Krebbel <krebbel at linux dot vnet dot ibm dot com>
- To: gcc-patches at gcc dot gnu dot org
- Date: Fri, 22 May 2015 09:52:35 +0200
- Subject: Re: [PATCH 02/13] optabs: Fix vec_perm -> V16QI middle end lowering.
- Authentication-results: sourceware.org; auth=none
- References: <1431350621-21405-1-git-send-email-krebbel at linux dot vnet dot ibm dot com> <1431350621-21405-3-git-send-email-krebbel at linux dot vnet dot ibm dot com> <555A22E8 dot 20307 at redhat dot com> <555AF724 dot 5000901 at linux dot vnet dot ibm dot com> <555B4D3D dot 2070001 at redhat dot com>
On Tue, May 19, 2015 at 07:48:29AM -0700, Richard Henderson wrote:
> > Ok to apply with that change?
>
> Yes, thanks.
I've applied the following.
Bye,
-Andreas-
gcc/
* optabs.c (expand_vec_perm): Don't re-use SEL as target operand.
---
gcc/optabs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gcc/optabs.c b/gcc/optabs.c
index bd03fc1..bc19029 100644
--- a/gcc/optabs.c
+++ b/gcc/optabs.c
@@ -6796,11 +6796,11 @@ expand_vec_perm (machine_mode mode, rtx v0, rtx v1, rtx sel, rtx target)
machine_mode selmode = GET_MODE (sel);
if (u == 2)
sel = expand_simple_binop (selmode, PLUS, sel, sel,
- sel, 0, OPTAB_DIRECT);
+ NULL, 0, OPTAB_DIRECT);
else
sel = expand_simple_binop (selmode, ASHIFT, sel,
GEN_INT (exact_log2 (u)),
- sel, 0, OPTAB_DIRECT);
+ NULL, 0, OPTAB_DIRECT);
gcc_assert (sel != NULL);
/* Broadcast the low byte each element into each of its bytes. */
--
1.7.9.5