This is the mail archive of the gcc-patches@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]

Re: [C++] Fix __builtin_shuffle

From: Marc Glisse <marc dot glisse at inria dot fr>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: Jason Merrill <jason at redhat dot com>, gcc-patches at gcc dot gnu dot org
Date: Thu, 27 Jun 2013 18:10:03 +0200 (CEST)
Subject: Re: [C++] Fix __builtin_shuffle
References: <alpine dot DEB dot 2 dot 02 dot 1306091302380 dot 27498 at stedding dot saclay dot inria dot fr> <51CB558B dot 6090905 at redhat dot com> <alpine dot DEB dot 2 dot 10 dot 1306271344370 dot 26429 at stedding dot saclay dot inria dot fr> <51CC2F9D dot 908 at redhat dot com> <alpine dot DEB dot 2 dot 10 dot 1306271719330 dot 26429 at stedding dot saclay dot inria dot fr> <20130627153313 dot GE2336 at tucnak dot redhat dot com> <alpine dot DEB dot 2 dot 10 dot 1306271746331 dot 26429 at stedding dot saclay dot inria dot fr> <20130627160057 dot GF2336 at tucnak dot redhat dot com>

On Thu, 27 Jun 2013, Jakub Jelinek wrote:

On Thu, Jun 27, 2013 at 05:54:10PM +0200, Marc Glisse wrote:

I don't really see why, as I am still calling c_build_vec_perm_expr
in the same cases, just possibly not exactly with the same arguments
(they don't go through build_non_dependent_expr, but Jason seemed to
imply that it did not matter since we don't look too deep through
the arguments).


I guess you're right.  If the c_* routine doesn't mind C++ specific trees
and just cares about their types and pt.c then instantiates it, then it is
fine.

Even if both are fine, if you prefer the long version (safer, or moreuniform with the rest), please say so, I don't mind.


--
Marc Glisse

Follow-Ups:
- Re: [C++] Fix __builtin_shuffle
  - From: Marc Glisse

References:
- [C++] Fix __builtin_shuffle
  - From: Marc Glisse
- Re: [C++] Fix __builtin_shuffle
  - From: Jason Merrill
- Re: [C++] Fix __builtin_shuffle
  - From: Marc Glisse
- Re: [C++] Fix __builtin_shuffle
  - From: Jason Merrill
- Re: [C++] Fix __builtin_shuffle
  - From: Marc Glisse
- Re: [C++] Fix __builtin_shuffle
  - From: Jakub Jelinek
- Re: [C++] Fix __builtin_shuffle
  - From: Marc Glisse
- Re: [C++] Fix __builtin_shuffle
  - From: Jakub Jelinek

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