[PATCH, vectorizer]: Take3: Vectorize FP conversions

Uros Bizjak ubizjak@gmail.com
Sun Apr 22 16:17:00 GMT 2007


Dorit Nuzman wrote:

> I'm sorry for not raising the following question before (didn't occur to me
> until now): Do we really need to duplicate the vec_unpack optab for floats,
> rather than reusing the vec_unpacks for both ints and floats, just like
> optabs smul,smin,smax? (either way is fine with me, just a question of
> general optab policy in gcc)
>   
Hm, I never noticed that TYPE_UNSIGNED also operates correctly on float 
types.

Due to this, this revision includes your suggestion to name FP patterns 
vec_unpacks_*.

>> Index: gcc/testsuite/gcc.dg/vect/vect-float-extend-1.c
>> ===================================================================
>> --- gcc/testsuite/gcc.dg/vect/vect-float-extend-1.c   (revision 0)
>> +++ gcc/testsuite/gcc.dg/vect/vect-float-extend-1.c   (revision 0)
>> @@ -0,0 +1,40 @@
>> +/* { dg-require-effective-target vect_double } */
>>     
> ...
>   
>> +
>> +/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect"
>> { target i?86-*-* x86_64-*-* } } } */
>>     
>
> I would xfail on spu instead.
>   

This change is also included in this patch.

Thanks for your comments!

The patch was bootstrapped on x86_64-pc-linux gnu and regression tested 
for all default languages.

OK for mainline (the patch need middle-end approval).

Uros.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: vect-conv-3.diff
Type: text/x-patch
Size: 20163 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20070422/a424b2a8/attachment.bin>


More information about the Gcc-patches mailing list