expansion of vector shifts...

David Miller davem@davemloft.net
Wed Feb 13 18:16:00 GMT 2013


From: Richard Biener <richard.guenther@gmail.com>
Date: Wed, 13 Feb 2013 12:15:13 +0100

> On Tue, Feb 12, 2013 at 11:31 PM, David Miller <davem@davemloft.net> wrote:
>> Maybe what we really mean to do here is check both op1 and SUBREG_REG
>> (op1) against SCALAR_INT_MODE_P instead of INTEGRAL_MODE_P?
> 
> Yes.

Ok, I'll commit this after doing some regstraps, thanks.

>> Something like this:
>>
>> gcc/
>>
>> 2013-02-12  David S. Miller  <davem@davemloft.net>
>>
>>         * expmed.c (expand_shift_1): Only strip scalar integer subregs.
>>
>> diff --git a/gcc/expmed.c b/gcc/expmed.c
>> index 4a6ddb0..954a360 100644
>> --- a/gcc/expmed.c
>> +++ b/gcc/expmed.c
>> @@ -2116,8 +2116,8 @@ expand_shift_1 (enum tree_code code, enum machine_mode mode, rtx shifted,
>>                        % GET_MODE_BITSIZE (mode));
>>        else if (GET_CODE (op1) == SUBREG
>>                && subreg_lowpart_p (op1)
>> -              && INTEGRAL_MODE_P (GET_MODE (SUBREG_REG (op1)))
>> -              && INTEGRAL_MODE_P (GET_MODE (op1)))
>> +              && SCALAR_INT_MODE_P (GET_MODE (SUBREG_REG (op1)))
>> +              && SCALAR_INT_MODE_P (GET_MODE (op1)))
>>         op1 = SUBREG_REG (op1);
>>      }
>>
>>
>>
> 



More information about the Gcc-patches mailing list