On 10/30/2015 02:23 PM, Cesar Philippidis wrote:
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.
Oh, there's something specific to the offloading support that needs
context.h, it's not the ptx port. Duh. Anyway, it'd still be good to
get standard builds of the nvptx backend into config-list.mk.
Not sure how painful it'd be to add the offloading path.
This is a great example of why we're trying to minimize/eliminate
conditional compilation :-)
jeff
