This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug c++/51033] generic vector subscript and shuffle support was not added to C++


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

Ramana Radhakrishnan <ramana at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ramana at gcc dot gnu.org

--- Comment #25 from Ramana Radhakrishnan <ramana at gcc dot gnu.org> 2012-06-13 15:14:16 UTC ---
(In reply to comment #19)
> Created attachment 27217 [details]
> shuffle
> 
> With this patch, g++ passes the few __builtin_shuffle tests I tried, and
> generates generic code for non-constant indexes and special code for constant
> indexes. I don't really know what to do about the testsuite. The tests exercise
> the backend a lot, and it probably doesn't make sense to run everything with
> both gcc and g++. But we still want to test that g++ accepts the syntax, and
> maybe even that it handles constants well.
> 
> Content of the patch:
> - move c_build_vec_perm_expr to c-common and condition the maybe_const stuff to
> the dialect
> - adapt the C RID_BUILTIN_SHUFFLE parser code to the C++ FE (the 2 are
> different enough that it isn't easy to share)
> - remove the C_ONLY tag from __builtin_shuffle
> 
> As usual, my limited knowledge of the compiler means I may have missed
> fundamental things.

Are you planning on submitting the shuffle patch as well ?


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]