Re: [PATCH 4/4] Modifications to the testsuite
Hi, On Wed, Oct 02 2019, Andreas Schwab wrote: > FAIL: gcc.dg/ipa/ipa-sra-19.c (test for excess errors) > Excess errors: > /daten/gcc/gcc-20191001/gcc/testsuite/gcc.dg/ipa/ipa-sra-19.c:19:3: error: > AltiVec argument passed to unprototyped function > /daten/gcc/gcc-20191001/gcc/testsuite/gcc.dg/ipa/ipa-sra-19.c:17:12: warning: > GCC vector returned by reference: non-standard ABI extension with no > compatibility guarantee [-Wpsabi] Some context like the target architecture would be appreciated. If it is power then the same has been reported as PR 91842 (which I fixed by skipping the test on power). Martin
Re: [PATCH 4/4] Modifications to the testsuite
On Wed, Oct 02, 2019 at 09:29:22PM +0200, Andreas Schwab wrote: > FAIL: gcc.dg/ipa/ipa-sra-19.c (test for excess errors) > Excess errors: > /daten/gcc/gcc-20191001/gcc/testsuite/gcc.dg/ipa/ipa-sra-19.c:19:3: error: > AltiVec argument passed to unprototyped function > /daten/gcc/gcc-20191001/gcc/testsuite/gcc.dg/ipa/ipa-sra-19.c:17:12: warning: > GCC vector returned by reference: non-standard ABI extension with no > compatibility guarantee [-Wpsabi] Yeah. Many tests add -Wno-psabi to shut these up; is that good here? Segher
Re: [PATCH 4/4] Modifications to the testsuite
FAIL: gcc.dg/ipa/ipa-sra-19.c (test for excess errors) Excess errors: /daten/gcc/gcc-20191001/gcc/testsuite/gcc.dg/ipa/ipa-sra-19.c:19:3: error: AltiVec argument passed to unprototyped function /daten/gcc/gcc-20191001/gcc/testsuite/gcc.dg/ipa/ipa-sra-19.c:17:12: warning: GCC vector returned by reference: non-standard ABI extension with no compatibility guarantee [-Wpsabi] Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF73 E780 A9DA AEC1 "And now for something completely different."
Re: [PATCH 4/4] Modifications to the testsuite
> This are all modifications to the testsuite required to get to the > state described in the cover letter of the entire IPA-SRA > patch-series. Please note that ipa/ipa-sra-2.c and ipa/ipa-sra-6.c > should actually be svn rm-ed instead as they try to invoke > functionality that the new IPA-SRA does not have (splitting aggregates > passed by reference into individual bits passed by reference). For > more information, see the cover letter of the whole IPA-SRA patch-set. > > This patch has already been approved by Jeff, but I'm re-sending it > for completeness. There has actually been a conflict in the options > of an LTO testcase, that is the only change compared to the previous > submission. > > Martin > > 2019-08-20 Martin Jambor > > * g++.dg/ipa/pr81248.C: Adjust dg-options and dump-scan. > * gcc.dg/ipa/ipa-sra-1.c: Likewise. > * gcc.dg/ipa/ipa-sra-10.c: Likewise. > * gcc.dg/ipa/ipa-sra-11.c: Likewise. > * gcc.dg/ipa/ipa-sra-3.c: Likewise. > * gcc.dg/ipa/ipa-sra-4.c: Likewise. > * gcc.dg/ipa/ipa-sra-5.c: Likewise. > * gcc.dg/ipa/ipacost-2.c: Disable ipa-sra. > * gcc.dg/ipa/ipcp-agg-9.c: Likewise. > * gcc.dg/ipa/pr78121.c: Adjust scan pattern. > * gcc.dg/ipa/vrp1.c: Likewise. > * gcc.dg/ipa/vrp2.c: Likewise. > * gcc.dg/ipa/vrp3.c: Likewise. > * gcc.dg/ipa/vrp7.c: Likewise. > * gcc.dg/ipa/vrp8.c: Likewise. > * gcc.dg/noreorder.c: use noipa attribute instead of noinline. > * gcc.dg/ipa/20040703-wpa.c: New test. > * gcc.dg/ipa/ipa-sra-12.c: New test. > * gcc.dg/ipa/ipa-sra-13.c: Likewise. > * gcc.dg/ipa/ipa-sra-14.c: Likewise. > * gcc.dg/ipa/ipa-sra-15.c: Likewise. > * gcc.dg/ipa/ipa-sra-16.c: Likewise. > * gcc.dg/ipa/ipa-sra-17.c: Likewise. > * gcc.dg/ipa/ipa-sra-18.c: Likewise. > * gcc.dg/ipa/ipa-sra-19.c: Likewise. > * gcc.dg/ipa/ipa-sra-20.c: Likewise. > * gcc.dg/ipa/ipa-sra-21.c: Likewise. > * gcc.dg/ipa/ipa-sra-22.c: Likewise. > * gcc.dg/sso/ipa-sra-1.c: Likewise. > * g++.dg/ipa/ipa-sra-2.C: Likewise. > * g++.dg/ipa/ipa-sra-3.C: Likewise. > * gcc.dg/tree-ssa/ipa-cp-1.c: Make return value used. > * g++.dg/ipa/devirt-19.C: Add missing return, add -fipa-cp-clone > option. > * g++.dg/lto/devirt-19_0.C: Add -fipa-cp-clone option. > > * gcc.dg/ipa/ipa-sra-2.c: Mark to be removed. > * gcc.dg/ipa/ipa-sra-6.c: Likewise. OK, thanks! Honza
Re: [PATCH 4/4] Modifications to the testsuite
On 7/23/19 10:16 AM, Martin Jambor wrote: > This are all modifications to the testsuite required to get to the > state described in the cover letter of the entire IPA-SRA > patch-series. Please note that ipa/ipa-sra-2.c and ipa/ipa-sra-6.c > should actually be svn rm-ed instead as they try to invoke > functionality that the new IPA-SRA does not have (splitting aggregates > passed by reference into individual bits passed by reference). For > more information, see the cover letter of the whole IPA-SRA patch-set. > > Martin > > 2019-07-23 Martin Jambor > > * g++.dg/ipa/pr81248.C: Adjust dg-options and dump-scan. > * gcc.dg/ipa/ipa-sra-1.c: Likewise. > * gcc.dg/ipa/ipa-sra-10.c: Likewise. > * gcc.dg/ipa/ipa-sra-11.c: Likewise. > * gcc.dg/ipa/ipa-sra-3.c: Likewise. > * gcc.dg/ipa/ipa-sra-4.c: Likewise. > * gcc.dg/ipa/ipa-sra-5.c: Likewise. > * gcc.dg/ipa/ipacost-2.c: Disable ipa-sra. > * gcc.dg/ipa/ipcp-agg-9.c: Likewise. > * gcc.dg/ipa/pr78121.c: Adjust scan pattern. > * gcc.dg/ipa/vrp1.c: Likewise. > * gcc.dg/ipa/vrp2.c: Likewise. > * gcc.dg/ipa/vrp3.c: Likewise. > * gcc.dg/ipa/vrp7.c: Likewise. > * gcc.dg/ipa/vrp8.c: Likewise. > * gcc.dg/noreorder.c: use noipa attribute instead of noinline. > * gcc.dg/ipa/20040703-wpa.c: New test. > * gcc.dg/ipa/ipa-sra-12.c: New test. > * gcc.dg/ipa/ipa-sra-13.c: Likewise. > * gcc.dg/ipa/ipa-sra-14.c: Likewise. > * gcc.dg/ipa/ipa-sra-15.c: Likewise. > * gcc.dg/ipa/ipa-sra-16.c: Likewise. > * gcc.dg/ipa/ipa-sra-17.c: Likewise. > * gcc.dg/ipa/ipa-sra-18.c: Likewise. > * gcc.dg/ipa/ipa-sra-19.c: Likewise. > * gcc.dg/ipa/ipa-sra-20.c: Likewise. > * gcc.dg/ipa/ipa-sra-21.c: Likewise. > * gcc.dg/ipa/ipa-sra-22.c: Likewise. > * gcc.dg/sso/ipa-sra-1.c: Likewise. > * g++.dg/ipa/ipa-sra-2.C: Likewise. > * g++.dg/ipa/ipa-sra-3.C: Likewise. > * gcc.dg/tree-ssa/ipa-cp-1.c: Make return value used. > * g++.dg/ipa/devirt-19.C: Add missing return, add -fipa-cp-clone > option. > * g++.dg/lto/devirt-19_0.C: Add -fipa-cp-clone option. > > * gcc.dg/ipa/ipa-sra-2.c: Mark to be removed. > * gcc.dg/ipa/ipa-sra-6.c: Likewise. This is fine once the prereqs are approved. jeff