This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PING] [PING] Make the OpenACC C++ acc_on_device wrapper "always inline"
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Thomas Schwinge <thomas at codesourcery dot com>
- Cc: Nathan Sidwell <nathan at acm dot org>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 6 Jun 2017 08:58:21 +0200
- Subject: Re: [PING] [PING] Make the OpenACC C++ acc_on_device wrapper "always inline"
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx03.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx03.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=jakub at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 9BB6280F6B
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 9BB6280F6B
- References: <56327E51.1090507@acm.org> <568AB69F.4020502@acm.org> <568D3764.1050104@acm.org> <5632B856.4050509@acm.org> <87h90ba8u8.fsf@euler.schwinge.homeip.net> <87wp8yv7xq.fsf@hertz.schwinge.homeip.net> <871sqx1v43.fsf@euler.schwinge.homeip.net>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Jun 06, 2017 at 08:35:40AM +0200, Thomas Schwinge wrote:
> > > commit 9cc3a384c17e9f692f7864c604d2e2f9fbf0bac9
> > > Author: Thomas Schwinge <thomas@codesourcery.com>
> > > Date: Tue May 23 13:21:14 2017 +0200
> > >
> > > Make the OpenACC C++ acc_on_device wrapper "always inline"
> > >
> > > libgomp/
> > > * openacc.h [__cplusplus] (acc_on_device): Mark as "always
> > > inline".
> > > * testsuite/libgomp.oacc-c-c++-common/acc-on-device-2.c: Remove
> > > file; test cases already present...
> > > * testsuite/libgomp.oacc-c-c++-common/acc_on_device-1.c: ... in
> > > this file. Update.
> > > * testsuite/libgomp.oacc-c-c++-common/acc-on-device.c: Remove
> > > file; test cases now present...
> > > * testsuite/libgomp.oacc-c-c++-common/acc_on_device-2.c: ... in
> > > this new file.
> > > * testsuite/libgomp.oacc-c-c++-common/parallel-dims.c: Update.
I don't like this very much.
Can't you instead just turn the builtin into BT_FN_INT_VAR and diagnose
during folding if it has no or 2+ arguments or if the argument is not type
compatible with int?
Jakub