On Wed, Jan 14, 2015 at 1:14 PM, Ilya Tocar <tocarip.in...@gmail.com> wrote:
> On 14 Jan 12:36, Uros Bizjak wrote:
>> On Wed, Jan 14, 2015 at 12:18 PM, Ilya Tocar <tocarip.in...@gmail.com> wrote:
>> > Hi,
>> >
>> > This patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64387
>> > Which was caused by different in predicates between vec_unpacks_hi
>> > and vec_extract_hi.
>>
>> Why are vec_unpacks_hi_{v8sf,v16sf} expanders different than
>> vec_unpacks_hi_v4sf? I think that these should all be expand in the
>> same way, similar to vec_unpacks_hi_v4sf.
>>
> In v4sf case we use movhlps, which is not avalible in v{8,16}sf case.

I see. We are generating vextract<something> here, that has
"register_operand" for its operand 1 constraint.

Patch is OK.

Thanks,
Uros.

Reply via email to