------- Comment #3 from Emmanuel dot Thome at inria dot fr  2008-05-29 14:02 
-------
> Yes there is, the __builtin_ia32_psrlqi128 takes __m128 and not a vector 
> type. 

ok. I guess you mean __m128i, for that particular case.

It very much seems that the texinfo documentation for this part of gcc is to be
forgotten for good, then.

> Also really using any of these builtins directly here is wrong as you should 
> be
> using the SSE intrinsics instead.  _mm_srl_epi64 also take a __m128 and not a
> vector type.

Sure -- this I gathered from previous PR in the same area. Which leaves pending
the question of documenting ``right'' vs ``wrong'' ;-).


I've ranted some rubbish above -- the psrlq builtins and friends are still
completely there indeed, I just missed the fact that commit 123250 reverted
commit 123215, which I expected at first to have broken things.

Regards,

E.


-- 

Emmanuel dot Thome at inria dot fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|target                      |c


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36370

Reply via email to