This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [gomp4] Add tables generation
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Bernd Schmidt <bernds at codesourcery dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Ilya Verbin <iverbin at gmail dot com>, Michael Zolotukhin <michael dot v dot zolotukhin at gmail dot com>
- Date: Fri, 21 Mar 2014 16:20:53 +0100
- Subject: Re: [gomp4] Add tables generation
- Authentication-results: sourceware.org; auth=none
- References: <532B1C45 dot 9020308 at codesourcery dot com> <20140320185619 dot GN1817 at tucnak dot redhat dot com> <532C5729 dot 5050101 at codesourcery dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Fri, Mar 21, 2014 at 04:13:45PM +0100, Bernd Schmidt wrote:
> On 03/20/2014 07:56 PM, Jakub Jelinek wrote:
> >When we were discussing the design last year, my strong preference was that
> >either this lives in some other crt object that mkoffload/linker plugin adds
> >to link, or that it would be completely mkoffload synthetized.
>
> mkoffload is only concerned with generating target images. These
> fragments are for the host tables.
>
> How's this? It moves everything to ompbegin.o/ompend.o and only
> links in these files if we have produced at least one target offload
> image.
I'd call the files crtompbegin.o/crtompend.o instead.
And, what is the exact reason why you are using protected visibility rather
than hidden?
Also, supposedly if you've used section names without . in them, the linker
itself would provide the symbols automatically and you wouldn't actually
need begin/end, but just one object that would reference the linker created
symbols. Just use say __gnu_offload_whatever__ or similar section names.
As for the __OPENMP_TARGET__ header format, that can be certainly resolved
later on.
Jakub