From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 100014 invoked by alias); 24 Feb 2020 12:33:48 -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 100004 invoked by uid 89); 24 Feb 2020 12:33:47 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-8.3 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_2,GIT_PATCH_3,RCVD_IN_DNSWL_NONE,SCC_10_SHORT_WORD_LINES,SCC_5_SHORT_WORD_LINES,SPF_PASS autolearn=ham version=3.3.1 spammy=H*RU:209.85.167.195, HX-Spam-Relays-External:209.85.167.195 X-HELO: mail-oi1-f195.google.com Received: from mail-oi1-f195.google.com (HELO mail-oi1-f195.google.com) (209.85.167.195) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 24 Feb 2020 12:33:45 +0000 Received: by mail-oi1-f195.google.com with SMTP id a22so8735344oid.13 for ; Mon, 24 Feb 2020 04:33:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=QPauw2TmQOwDXibSWyuwscqXkHPTwYIgvE0jziUInTw=; b=RKrlMQytH3AUAGZhxgim5we+Uyq64RMTIj4cWGvfa+QHB13t/AiU508l2fB4edMESq SwaXGZQCuLDaiZrxY150ZP6Z8sV4mYXyxaiFHScMZ4aJ6VsYRYMSawTgE0fhNL22Ax15 Ff8t6srZKj3IvIlJQ3yisQw1z2N+bqDoRqDu0Fp8uO9qGhUHDyqt3Hk6ATESv4yzpC7e mKO7P1nLa6OjDB5dSi6hUVyLKnSQIDqpAzsG9xm2ZvGK/TxFvjTgK/XZ59J1WKpZDnsn tG7ExKA5kb7Ahfq+WDq9FILTx/qyRx+0f7ANoUbg56Wye04ZpnkKKW3bM3ggYuiYQ21O FWmw== MIME-Version: 1.0 References: <20200224114950.GE2155@tucnak> In-Reply-To: <20200224114950.GE2155@tucnak> From: Christophe Lyon Date: Mon, 24 Feb 2020 12:33:00 -0000 Message-ID: Subject: Re: [PATCH] testsuite: Fix recently added ipa testcases [PR93763] To: Jakub Jelinek Cc: Jeff Law , Martin Jambor , Feng Xue OS , Jan Hubicka , "gcc-patches@gcc.gnu.org" Content-Type: text/plain; charset="UTF-8" X-IsSubscribed: yes X-SW-Source: 2020-02/txt/msg01288.txt.bz2 On Mon, 24 Feb 2020 at 12:50, Jakub Jelinek wrote: > > On Mon, Feb 24, 2020 at 11:09:58AM +0100, Christophe Lyon wrote: > > I'm seeing errors with the new testcase: > > end of struct or union > > /gcc/testsuite/gcc.dg/ipa/pr93763.c:8:1: warning: data definition has > > no type or storage class > ... > > Seems the test has been badly reduced (if the original doesn't emit > warnings, it is always better in the reduction script avoid introducing new > ones). > Also, the g++.dg/ipa/ test fails with -std=c++98 because it is written in > C++11. > > Fixed thusly, ok for trunk? Works for me, thanks > > 2020-02-24 Jakub Jelinek > > PR ipa/93763 > * gcc.dg/ipa/pr93763.c: Adjust the test so that it compiles without > warnings and still ICEs before the ipa-cp.c fix. > * g++.dg/ipa/pr93763.C: Require c++11 effective target. > > --- gcc/testsuite/gcc.dg/ipa/pr93763.c.jj 2020-02-24 12:44:09.626326146 +0100 > +++ gcc/testsuite/gcc.dg/ipa/pr93763.c 2020-02-24 12:40:48.707320934 +0100 > @@ -3,44 +3,48 @@ > > typedef struct a a; > struct a { > - a *b > + a *b; > } d; > -e, k, ah, al; > -f(aa) { > +int e, k, ah, al; > +void h(void); > +void > +f(aa) int aa; { > if (aa & 1) > goto g; > f(aa | 2); > g: > h(); > } > +void i(); > +void > l() { > - { > f(072); > i(e, d, 92); > - } > } > +void > ag() { > - { i(e, d, 36); } > + i(e, d, 36); > } > +void j(); > +void > ai(a *m, a *n, unsigned aa) { > f(aa); > j(k, l, ah, 1); > } > +void > j(int c, a m, int aj, int aa) { > int ak = aa; > - { i(e, d, ak); } > + i(e, d, ak); > } > +void > i(int c, a *m, unsigned aa) { > - { > - { i(c, (*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*( > + i(c, (*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*( > *(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*( > *(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*(*m).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b) > .b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b) > .b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b) > .b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b) > .b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b).b, 0); > - } > - } > int am = aa; > - ai(ag, al, am); > + ai((a *) (void *) ag, (a *) (__INTPTR_TYPE__) al, am); > } > --- gcc/testsuite/g++.dg/ipa/pr93763.C.jj 2020-02-21 21:54:56.616348316 +0100 > +++ gcc/testsuite/g++.dg/ipa/pr93763.C 2020-02-24 12:35:02.061484741 +0100 > @@ -1,4 +1,4 @@ > -/* { dg-do compile } */ > +/* { dg-do compile { target c++11 } } */ > /* { dg-options "-O3" } */ > > struct search_param { > > > Jakub >