From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13120 invoked by alias); 9 Feb 2016 11:26:45 -0000 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 Received: (qmail 13108 invoked by uid 89); 9 Feb 2016 11:26:44 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.2 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mail-yk0-f171.google.com Received: from mail-yk0-f171.google.com (HELO mail-yk0-f171.google.com) (209.85.160.171) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Tue, 09 Feb 2016 11:26:44 +0000 Received: by mail-yk0-f171.google.com with SMTP id z7so98658761yka.3 for ; Tue, 09 Feb 2016 03:26:43 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=JtxU6F3rjZQxLw42lvFTeAejGBwbJnNJQKYv4hmz3Wg=; b=IzpXFNTB7bjU8AhZKtdCrK0SpXAvWKrkLsQuzCWz+Q+EEJNmRUyjpcr3CAhwX5p+pq EhB7tC2ylUD8EhFV8Vaz2iCJdBAemzP5fVBkLEStcV/oXiuG0AmXM56S+KHDzp1+bHVh s9ne2r0SZKDbmcTVZd/YTX9iLy7bOwncQppG4e1q350xXEqsG19ihAYZzQ3oIwMF+oOS o3A/B3KR/jctUXmVTwXgYtSFLsqoNck21QmlrhY/djVdCpJVn0Duz1qkcT6uJsgijavI u9wiocyS07gQJFsVpjii0saDKIfGWDCfzCQhrl1r14Gm7mCti6Sg9Ycu0ZPgGi8RUO/C d/VQ== X-Gm-Message-State: AG10YOTeZEDAld6a/y492KB4oujyulxPXLms7JP2I0+z5EwJA8tGxWP6tpcnhUWnW9Urm6nYq8OpZ4E0uo+jRw== MIME-Version: 1.0 X-Received: by 10.37.21.5 with SMTP id 5mr16924583ybv.22.1455017201854; Tue, 09 Feb 2016 03:26:41 -0800 (PST) Received: by 10.37.223.11 with HTTP; Tue, 9 Feb 2016 03:26:41 -0800 (PST) In-Reply-To: <1455017097-24724-1-git-send-email-james.greenhalgh@arm.com> References: <1455017097-24724-1-git-send-email-james.greenhalgh@arm.com> Date: Tue, 09 Feb 2016 11:26:00 -0000 Message-ID: Subject: Re: [Patch] Gate vect-mask-store-move-1.c correctly, and actually output the dump From: Richard Biener To: James Greenhalgh Cc: GCC Patches , nd@arm.com, Uros Bizjak , =?UTF-8?B?0JjQu9GM0Y8g0K3QvdC60L7QstC40Yc=?= , Igor Zamyatin , ysrumyan@gmail.com Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2016-02/txt/msg00615.txt.bz2 On Tue, Feb 9, 2016 at 12:24 PM, James Greenhalgh wrote: > > On Mon, Feb 08, 2016 at 03:24:14PM +0100, Richard Biener wrote: >> On Mon, Feb 8, 2016 at 2:40 PM, James Greenhalgh >> wrote: >> > On Mon, Feb 08, 2016 at 04:29:31PM +0300, Yuri Rumyantsev wrote: >> >> Hi James, >> >> >> >> Thanks for reporting this issue. >> >> I prepared slightly different patch since we don't need to add >> >> tree-vect dump option - it is on by default for all tests in /vect >> >> directory. >> > >> > Hm, I added that line as my test runs were showing: >> > >> > UNRESOLVED: gcc.dg/vect/vect-mask-store-move-1.c: dump file does not exist >> > >> > I would guess the explicit >> > >> > /* { dg-options "-O3" } */ >> > >> > is clobbering the vect.exp setup of flags? >> >> Yes. Use { dg-additional-options "-O3" } instead. > > I don't see why this test needs anything more than the default vect > options anyway... In which case, the patch would look like this. > > Tested on x86-64 where the test passes, and on AArch64 where it is > correctly skipped. > > OK? Ok. Richard. > Thanks, > James > > --- > 2016-02-09 James Greenhalgh > > * gcc.dg/vect/vect-mask-store-move-1.c: Drop dg-options directive, > gate check on x86_64/i?86. >