This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] New backend header reduction
- From: Cesar Philippidis <cesar at codesourcery dot com>
- To: Andrew MacLeod <amacleod at redhat dot com>, Jeff Law <law at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 30 Oct 2015 13:23:59 -0700
- Subject: Re: [patch] New backend header reduction
- Authentication-results: sourceware.org; auth=none
- References: <562A8987 dot 2030508 at redhat dot com> <5633AF4E dot 6040105 at codesourcery dot com> <5633B266 dot 1030008 at redhat dot com> <5633D100 dot 4070509 at redhat dot com>
On 10/30/2015 01:20 PM, Andrew MacLeod wrote:
> On 10/30/2015 02:09 PM, Andrew MacLeod wrote:
>> On 10/30/2015 01:56 PM, Cesar Philippidis wrote:
>>> On 10/23/2015 12:24 PM, Jeff Law wrote:
>>>> On 10/23/2015 10:53 AM, Andrew MacLeod wrote:
>>>>
>>> There's a little bit of fallout with this patch when building an
>>> offloaded compiler for openacc. It looks like cgraph.c needs to include
>>> context.h and varpool.c needs context.h and omp-low.h. There's a couple
>>> of ifdef ENABLE_OFFLOADING which may have gone undetected with your
>>> script.
>> If they are defined on the command line or some other way I couldn't
>> see with the targets I built, then that is the common case when that
>> happens. I don't think I did any openacc builds. OR maybe I need
>> to add nvptx to my coverage builds. Perhaps that is best.
>>> I've bootstrapped the attached patch for an nvptx/x86_64-linux target.
>>> I'm still testing that toolchain. If the testing comes back clean, is
>>> this patch OK for trunk?
> Ah, I see. there is no nvptx target in config-list.mk, so it never got
> covered.
Yeah, you need to build two separate compilers. Thomas posted some
directions here <https://gcc.gnu.org/wiki/Offloading>. You could
probably reproduce it with openmp and Intel's MIC emulation target too.
Cesar