From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7763 invoked by alias); 19 Aug 2010 18:54:34 -0000 Received: (qmail 7742 invoked by uid 22791); 19 Aug 2010 18:54:32 -0000 X-SWARE-Spam-Status: No, hits=-1.7 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,SPF_HELO_PASS,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from smtp-out.google.com (HELO smtp-out.google.com) (216.239.44.51) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 19 Aug 2010 18:54:25 +0000 Received: from wpaz21.hot.corp.google.com (wpaz21.hot.corp.google.com [172.24.198.85]) by smtp-out.google.com with ESMTP id o7JIsNXg016977 for ; Thu, 19 Aug 2010 11:54:23 -0700 Received: from vws11 (vws11.prod.google.com [10.241.21.139]) by wpaz21.hot.corp.google.com with ESMTP id o7JIsMCv026075 for ; Thu, 19 Aug 2010 11:54:22 -0700 Received: by vws11 with SMTP id 11so2218526vws.22 for ; Thu, 19 Aug 2010 11:54:22 -0700 (PDT) MIME-Version: 1.0 Received: by 10.220.71.136 with SMTP id h8mr87302vcj.275.1282243911972; Thu, 19 Aug 2010 11:51:51 -0700 (PDT) Received: by 10.220.198.193 with HTTP; Thu, 19 Aug 2010 11:51:51 -0700 (PDT) In-Reply-To: <20100819181529.GA22089@caradoc.them.org> References: <87eiduu7pq.fsf@basil.nowhere.org> <20100819181529.GA22089@caradoc.them.org> Date: Thu, 19 Aug 2010 19:33:00 -0000 Message-ID: Subject: Re: define MMX/SSE intrinsics without requiring cmdline isa flags From: Xinliang David Li To: Xinliang David Li , "H.J. Lu" , Andi Kleen , GCC Patches Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-System-Of-Record: true X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2010-08/txt/msg01497.txt.bz2 On Thu, Aug 19, 2010 at 11:15 AM, Daniel Jacobowitz wrote: > On Thu, Aug 19, 2010 at 10:36:54AM -0700, Xinliang David Li wrote: >> You have not comment on why you think the patch is a kludge --- though >> it does not solve the problem of mixing regions with different target >> options in the same function, it is an independent patch on top of the >> per function target attribute that will make people's life much easier >> (all it does is predefine all isa builtins and 'unguard' the intrinsic >> wrapper defs). =A0It makes your second example in >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D39840 work when merging >> all sources into one file (after adding appropriate target attribute). > > Those macros are not used just by a GCC header file. =A0They're used by > user code too. =A0This seems like an obvious way to break a whole lot > of programs. I hear you -- this is a valid concern. Thanks, David > > -- > Daniel Jacobowitz > CodeSourcery >