Re: [PATCH 1/4] Remove build dependence on HSA run-time

2021-03-25 Thread Thomas Schwinge
Hi! On 2021-01-19T12:37:56+0100, Martin Jambor wrote: > On Thu, Jan 14 2021, Thomas Schwinge wrote: >> I'm raising here an issue with HSA libgomp plugin code changes from a >> while ago. While HSA is now no longer relevant for GCC master branch, >> the same code has also been copied into the GCN

Re: [PATCH 1/4] Remove build dependence on HSA run-time

2021-01-19 Thread Martin Liška
On 1/19/21 12:37 PM, Martin Jambor wrote: Right. From what I can tell at the moment, which is not much, the idea was to be able to load it even from a non-standard path and specify that path at configure time. If people think that is not useful and is actually harmful, I guess it can go. And

Re: [PATCH 1/4] Remove build dependence on HSA run-time

2021-01-19 Thread Martin Jambor
Hi Thomas, On Thu, Jan 14 2021, Thomas Schwinge wrote: > Hi! > > I'm raising here an issue with HSA libgomp plugin code changes from a > while ago. While HSA is now no longer relevant for GCC master branch, > the same code has also been copied into the GCN libgomp plugin. > > This is commit b8d89

Re: [PATCH 1/4] Remove build dependence on HSA run-time

2021-01-14 Thread Thomas Schwinge
Hi! I'm raising here an issue with HSA libgomp plugin code changes from a while ago. While HSA is now no longer relevant for GCC master branch, the same code has also been copied into the GCN libgomp plugin. This is commit b8d89b03db5f212919e4571671ebb4f5f8b1e19d (r242749) "Remove build dependen

Re: [PATCH 1/4] Remove build dependence on HSA run-time

2016-11-22 Thread Jakub Jelinek
On Tue, Nov 22, 2016 at 02:27:44PM +0100, Martin Jambor wrote: > I have basically copied what libgfortran did, with additional checking > for HAVE_UNISTD_H when attempting to implement secure_getenv in its > absence (which is maybe unnecessary but should not do any harm) and I > also needed to add

Re: [PATCH 1/4] Remove build dependence on HSA run-time

2016-11-22 Thread Martin Jambor
Hi, On Fri, Nov 18, 2016 at 11:23:10AM +0100, Jakub Jelinek wrote: > On Sun, Nov 13, 2016 at 08:02:41PM +0100, Martin Jambor wrote: > > @@ -143,6 +240,12 @@ init_enviroment_variables (void) > > suppress_host_fallback = true; > >else > > suppress_host_fallback = false; > > + > > + hs

Re: [PATCH 1/4] Remove build dependence on HSA run-time

2016-11-18 Thread Jakub Jelinek
On Sun, Nov 13, 2016 at 08:02:41PM +0100, Martin Jambor wrote: > @@ -143,6 +240,12 @@ init_enviroment_variables (void) > suppress_host_fallback = true; >else > suppress_host_fallback = false; > + > + hsa_runtime_lib = getenv ("HSA_RUNTIME_LIB"); > + if (hsa_runtime_lib == NULL) > +