From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1782 invoked by alias); 18 Jan 2013 19:32:51 -0000 Received: (qmail 1769 invoked by uid 22791); 18 Jan 2013 19:32:50 -0000 X-SWARE-Spam-Status: No, hits=-5.2 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,KHOP_RCVD_TRUST,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE X-Spam-Check-By: sourceware.org Received: from mail-vc0-f172.google.com (HELO mail-vc0-f172.google.com) (209.85.220.172) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 18 Jan 2013 19:32:44 +0000 Received: by mail-vc0-f172.google.com with SMTP id l6so2939006vcl.31 for ; Fri, 18 Jan 2013 11:32:44 -0800 (PST) X-Received: by 10.52.98.73 with SMTP id eg9mr9517682vdb.18.1358537564034; Fri, 18 Jan 2013 11:32:44 -0800 (PST) MIME-Version: 1.0 Received: by 10.58.215.38 with HTTP; Fri, 18 Jan 2013 11:32:03 -0800 (PST) In-Reply-To: <20130117235952.GX7269@tucnak.redhat.com> References: <20130117235952.GX7269@tucnak.redhat.com> From: Steven Bosscher Date: Fri, 18 Jan 2013 19:32:00 -0000 Message-ID: Subject: Re: [patch] PR inline-asm/55934 To: Jakub Jelinek Cc: Vladimir Makarov , GCC Patches Content-Type: text/plain; charset=ISO-8859-1 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: 2013-01/txt/msg00978.txt.bz2 On Fri, Jan 18, 2013 at 12:59 AM, Jakub Jelinek wrote: > On Fri, Jan 18, 2013 at 12:45:06AM +0100, Steven Bosscher wrote: >> --- testsuite/gcc.target/i386/pr55934.c (revision 0) >> +++ testsuite/gcc.target/i386/pr55934.c (revision 0) >> @@ -0,0 +1,10 @@ >> +/* PR inline-asm/55934 */ >> +/* { dg-do compile } */ >> +/* { dg-require-effective-target sse } */ > > Don't you need /* { dg-options "-msse" } */ ? dg-require-effective-target sse > itself doesn't add -msse option... Actually, I needed even more than that: /* { dg-options "-std=c99 -msse" } */ But I added that line after creating the diff, and forgot to update the test case in the patch :-) Ciao! Steven