On Fri, Dec 12, 2014 at 10:58:39AM +0100, Thomas Schwinge wrote:
> > --- /dev/null
> > +++ b/libgomp/libgomp_target.h
> > @@ -0,0 +1,44 @@
> > +/* Copyright (C) 2014 Free Software Foundation, Inc.
> > +
> > +   This file is part of the GNU OpenMP Library (libgomp).
> 
> > +#ifndef LIBGOMP_TARGET_H
> > +#define LIBGOMP_TARGET_H 1
> > +
> > +/* Type of offload target device.  */
> > +enum offload_target_type
> > +{
> > +  OFFLOAD_TARGET_TYPE_HOST,
> > +  OFFLOAD_TARGET_TYPE_INTEL_MIC
> > +};

Maybe this.

> > +/* Auxiliary struct, used for transferring a host-target address range 
> > mapping
> > +   from plugin to libgomp.  */
> > +struct mapping_table
> > +{
> > +  uintptr_t host_start;
> > +  uintptr_t host_end;
> > +  uintptr_t tgt_start;
> > +  uintptr_t tgt_end;
> > +};

But this IMHO doesn't belong to include/libgomp-constants.h, that
file is for stuff shared between the compiler and libgomp, while
the above is for communication between libgomp plugins and libgomp,
there is no point to put it outside of libgomp and the compiler
is not interested in this.

        Jakub

Reply via email to