From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-patches-return-344374-listarch-gcc-patches=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 7896 invoked by alias); 28 Jun 2013 10:34:12 -0000
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-patches.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-patches/>
List-Post: <mailto:gcc-patches@gcc.gnu.org>
List-Help: <mailto:gcc-patches-help@gcc.gnu.org>
Sender: gcc-patches-owner@gcc.gnu.org
Received: (qmail 7884 invoked by uid 89); 28 Jun 2013 10:34:11 -0000
X-Spam-SWARE-Status: No, score=-7.9 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_THREADED,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL,RP_MATCHES_RCVD autolearn=ham version=3.3.1
Received: from mail2-relais-roc.national.inria.fr (HELO mail2-relais-roc.national.inria.fr) (192.134.164.83)    by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Fri, 28 Jun 2013 10:34:09 +0000
Received: from ip-95.net-81-220-130.standre.rev.numericable.fr (HELO laptop-mg.local) ([81.220.130.95])  by mail2-relais-roc.national.inria.fr with ESMTP/TLS/DHE-RSA-AES256-SHA; 28 Jun 2013 12:34:07 +0200
Date: Fri, 28 Jun 2013 10:34:00 -0000
From: Marc Glisse <marc.glisse@inria.fr>
To: Jakub Jelinek <jakub@redhat.com>
cc: Jason Merrill <jason@redhat.com>, gcc-patches@gcc.gnu.org
Subject: Re: [C++] Fix __builtin_shuffle
In-Reply-To: <alpine.DEB.2.10.1306280909340.3911@laptop-mg.saclay.inria.fr>
Message-ID: <alpine.DEB.2.10.1306281232000.3911@laptop-mg.saclay.inria.fr>
References: <alpine.DEB.2.02.1306091302380.27498@stedding.saclay.inria.fr> <51CB558B.6090905@redhat.com> <alpine.DEB.2.10.1306271344370.26429@stedding.saclay.inria.fr> <20130628060050.GI2336@tucnak.redhat.com> <alpine.DEB.2.10.1306280909340.3911@laptop-mg.saclay.inria.fr>
User-Agent: Alpine 2.10 (DEB 1266 2009-07-14)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
X-SW-Source: 2013-06/txt/msg01586.txt.bz2

On Fri, 28 Jun 2013, Marc Glisse wrote:

> It seems better to do the following, so we still test for extra warnings:
>
> -template <class T> void g (T) {}
> -template <class T> auto g (T x) -> typename 
> enable_if<sizeof(__builtin_shuffle(x,x))!=2>::type {}
> +template <class T> void g (T const&) {}
> +template <class T> auto g (T const& x) -> typename 
> enable_if<sizeof(__builtin_shuffle(x,x))!=2>::type {}

I committed that as an obvious fix of my testcase.

2013-06-28  Marc Glisse  <marc.glisse@inria.fr>

 	PR c++/57509
 	* g++.dg/ext/pr57509.C: Pass vectors by reference to avoid warnings.


-- 
Marc Glisse