OMP builtins in offloading (was: [PATCH 1/4] Add mkoffload for Intel MIC)

Ilya Verbin iverbin@gmail.com
Fri Dec 26 15:49:00 GMT 2014


On 22 Dec 12:26, Jakub Jelinek wrote:
> On Mon, Dec 22, 2014 at 12:20:58PM +0100, Thomas Schwinge wrote:
> > What is the reason that you're adding -fopenmp here?  I assume it is that
> > otherwise you'd get tree streaming errors because of different builtins
> > configurations, like this?
> > 
> > If that is the "only" reason to add -fopenmp there, can we then instead
> > do the following?
> > 
> >  gcc/builtins.def                     | 8 ++++++--
> >  gcc/config/i386/intelmic-mkoffload.c | 1 -
> >  2 files changed, 6 insertions(+), 3 deletions(-)
> 
> If Ilya confirms that is the sole reason, I'm fine with this change.
> Please write ChangeLog entry for it.
> 
> 	Jakub

Ok to me.

Thanks,
  -- Ilya



More information about the Gcc-patches mailing list