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 inser
On 05/19/2015 01:41 AM, Andreas Krebbel wrote:
> On 05/18/2015 07:35 PM, Richard Henderson wrote:
>> On 05/11/2015 06:23 AM, Andreas Krebbel wrote:
>>> @@ -6784,14 +6784,18 @@ expand_vec_perm (machine_mode mode, rtx v0, rtx v1,
>>> rtx sel, rtx target)
>>> {
>>>/* Multiply each elemen
On 05/18/2015 07:35 PM, Richard Henderson wrote:
> On 05/11/2015 06:23 AM, Andreas Krebbel wrote:
>> @@ -6784,14 +6784,18 @@ expand_vec_perm (machine_mode mode, rtx v0, rtx v1,
>> rtx sel, rtx target)
>> {
>>/* Multiply each element by its byte size. */
>>machine_mode selmode
On 05/11/2015 06:23 AM, Andreas Krebbel wrote:
> @@ -6784,14 +6784,18 @@ expand_vec_perm (machine_mode mode, rtx v0, rtx v1,
> rtx sel, rtx target)
> {
>/* Multiply each element by its byte size. */
>machine_mode selmode = GET_MODE (sel);
> + /* We cannot re-use SEL as a
On 05/11/2015 07:23 AM, Andreas Krebbel wrote:
The current implementation re-uses the location of the selection
pattern to generate a new one. This fails if the pattern resides in a
read-only location. With the patch a new temporary register is
allocated for that purpose.
gcc/
* optabs
The current implementation re-uses the location of the selection
pattern to generate a new one. This fails if the pattern resides in a
read-only location. With the patch a new temporary register is
allocated for that purpose.
gcc/
* optabs.c (expand_vec_perm): Allocate a temp reg for the
