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.
For config-list.mk testing, we just need to be able to build cross
compilers for the given target -- the offloading bits wouldn't be
applicable here, unless the PTX backend inherently depends on them.
jeff
