This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug c++/51033] generic vector subscript and shuffle support was not added to C++
- From: "ramana at gcc dot gnu.org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: Wed, 13 Jun 2012 15:14:16 +0000
- Subject: [Bug c++/51033] generic vector subscript and shuffle support was not added to C++
- Auto-submitted: auto-generated
- References: <bug-51033-4@http.gcc.gnu.org/bugzilla/>
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 ?