From: Christophe Lyon <christophe.lyon@linaro.org>
To: Jakub Jelinek <jakub@redhat.com>
Cc: Jeff Law <law@redhat.com>, Martin Jambor <mjambor@suse.cz>,
Feng Xue OS <fxue@os.amperecomputing.com>,
Jan Hubicka <hubicka@ucw.cz>,
"gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>
Subject: Re: [PATCH] testsuite: Fix recently added ipa testcases [PR93763]
Date: Mon, 24 Feb 2020 12:33:00 -0000 [thread overview]
Message-ID: <CAKdteOZsPVU6SS8k0mk+0jShRg9=sG7_sHi-R37hZjwyFo0gFA@mail.gmail.com> (raw)
In-Reply-To: <20200224114950.GE2155@tucnak>
On Mon, 24 Feb 2020 at 12:50, Jakub Jelinek <jakub@redhat.com> 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 <jakub@redhat.com>
>
> 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
>
prev parent reply other threads:[~2020-02-24 12:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-18 15:12 [PATCH] Do not propagate self-dependent value (PR ipa/93763) Feng Xue OS
2020-02-21 17:59 ` Martin Jambor
2020-02-21 20:23 ` Jeff Law
2020-02-24 10:10 ` Christophe Lyon
2020-02-24 11:50 ` [PATCH] testsuite: Fix recently added ipa testcases [PR93763] Jakub Jelinek
2020-02-24 12:33 ` Christophe Lyon [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAKdteOZsPVU6SS8k0mk+0jShRg9=sG7_sHi-R37hZjwyFo0gFA@mail.gmail.com' \
--to=christophe.lyon@linaro.org \
--cc=fxue@os.amperecomputing.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=hubicka@ucw.cz \
--cc=jakub@redhat.com \
--cc=law@redhat.com \
--cc=mjambor@suse.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).