This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

RE: [PATCH] Fix OpenMP offloading to NVPTX and correct location of

From: Alexander Monakov <amonakov at ispras dot ru>
To: "Hahnfeld, Jonas" <Hahnfeld at itc dot rwth-aachen dot de>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "Jakub Jelinek (jakub at redhat dot com)" <jakub at redhat dot com>
Date: Mon, 28 Nov 2016 17:04:09 +0300 (MSK)
Subject: RE: [PATCH] Fix OpenMP offloading to NVPTX and correct location of
Authentication-results: sourceware.org; auth=none
References: <f227f56cc2924a4fa0d612e3006dbd0a@rwthex-w2-a.rwth-ad.de> <alpine.LNX.2.20.13.1611281539440.2229@monopod.intra.ispras.ru> <a26536242e5a47929c6ec7c9b2bf54e5@rwthex-w2-a.rwth-ad.de>

> > Thanks.  This is in the #else branch of '#ifdef HAVE_PTHREAD_AFFINITY_NP'
> > (so the ChangeLog entry should have [!HAVE_PTHREAD_AFFINITY_NP]) --
> > I'm curious why you don't have this defined.  One reason I can think of is, 
> > if you're building libgomp against musl libc

(I was mistaken here - musl libc does provide pthread_.*affinity_np entrypoints)

> I'm fine with any splitting or changes to my change as long as the problem is 
> fixed in the end ;-)

Applied as svn rev. 242918 and then deleted critical.c in the next commit.

Thanks.
Alexander

References:
- [PATCH] Fix OpenMP offloading to NVPTX and correct location of
  - From: Hahnfeld, Jonas
- Re: [PATCH] Fix OpenMP offloading to NVPTX and correct location of
  - From: Alexander Monakov
- RE: [PATCH] Fix OpenMP offloading to NVPTX and correct location of
  - From: Hahnfeld, Jonas

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]