This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [hsa, testsuite] Suppress hsa warnings in libgomp tests
- From: Jakub Jelinek <jakub at redhat dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 4 Mar 2016 17:04:31 +0100
- Subject: Re: [hsa, testsuite] Suppress hsa warnings in libgomp tests
- Authentication-results: sourceware.org; auth=none
- References: <20160226155957 dot GJ3094 at virgil dot suse dot cz> <20160226160756 dot GF3017 at tucnak dot redhat dot com> <20160301183918 dot GA2148 at virgil dot suse dot cz> <20160301184749 dot GH3017 at tucnak dot redhat dot com> <20160301214746 dot GA14713 at virgil dot suse dot cz> <20160301220643 dot GJ3017 at tucnak dot redhat dot com> <20160304152711 dot GN3068 at virgil dot suse dot cz> <20160304153129 dot GA3017 at tucnak dot redhat dot com> <20160304160134 dot GC2148 at virgil dot suse dot cz>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Mar 04, 2016 at 05:01:34PM +0100, Martin Jambor wrote:
> Not in the limited runs that I experimented with so far, but I
> certainly kept this possibility in mind too. If so, I would either
> set it back before invoking dg-finish or dismiss the whole idea.
>
> > > However, the C++ and Fortran cases use gfortran-dg-runtest to cycle
> > > through a set of torture options and I have not yet discovered the
> > > right magic variable to set (for example, adding -Wno-hsa to
> > > DG_TORTURE_OPTIONS elements does not work).
> > >
> > > I'm afraid I have spent way too much time on this already, so unless
> > > someone has any ideas, I'd suggest that we use the (already approved)
> > > name-changing gomp patch as it is. Or at least for C++ and Fortran.
> >
> > Do you have URL for what you refer to?
> >
>
> Sure, the patch has been posted here:
>
> https://gcc.gnu.org/ml/gcc-patches/2016-03/msg00071.html
For the g*.dg/gomp/, if you'd only move -Wno-hsa into the last argument
next to -fopenmp, how many tests would be affected? If not really many,
perhaps those could be changed to use dg-additional-options instead of
dg-options.
Jakub