Re: [PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter

2015-02-24 Thread Thomas Schwinge
t;https://gcc.gnu.org/PR64625>. > commit 4409d0129118479c1cd1adbcfa96316ac4e734b0 > Author: Thomas Schwinge > Date: Fri Jan 16 20:12:12 2015 +0100 > > [PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter. > > gcc/ > *

Re: [PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-01-16 Thread Jack Howarth
oing on? > > Anyway, as discussed in <https://gcc.gnu.org/PR64625>, I'd like to commit > this patch either way, OK? > > commit 4409d0129118479c1cd1adbcfa96316ac4e734b0 > Author: Thomas Schwinge > Date: Fri Jan 16 20:12:12 2015 +0100 > > [PR libgomp/64625

Re: [PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-01-16 Thread Ilya Verbin
On 16 Jan 19:23, Jack Howarth wrote: > As I read https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64625#c3, the > requirement for __OFFLOAD_TABLE__ was not longer present and the > residual usages of it just had to be removed. The weak symbol on > darwin is fragile and seems to trip up on the existing

Re: [PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-01-16 Thread Jack Howarth
As I read https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64625#c3, the requirement for __OFFLOAD_TABLE__ was not longer present and the residual usages of it just had to be removed. The weak symbol on darwin is fragile and seems to trip up on the existing code which produces undefined symbols for __

Re: [PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-01-16 Thread Ilya Verbin
On 16 Jan 18:22, Jack Howarth wrote: > On 86_64 Fedora 15, current gcc trunk only produces… > > nm libgcc_s.so.1 | grep OFF > 00215478 d _GLOBAL_OFFSET_TABLE_ > > and not __OFFLOAD_TABLE__, The libgcc_s.so.1 built on > x86_64-apple-darwin14 doesn't even contain the _GLOBAL_OFFSET_TABLE_

Re: [PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-01-16 Thread Jack Howarth
On 86_64 Fedora 15, current gcc trunk only produces… nm libgcc_s.so.1 | grep OFF 00215478 d _GLOBAL_OFFSET_TABLE_ and not __OFFLOAD_TABLE__, The libgcc_s.so.1 built on x86_64-apple-darwin14 doesn't even contain the _GLOBAL_OFFSET_TABLE_ symbol. On Fri, Jan 16, 2015 at 5:40 PM, Ilya Ver

Re: [PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-01-16 Thread Ilya Verbin
On 16 Jan 21:34, Thomas Schwinge wrote: > On Thu, 15 Jan 2015 21:20:07 +0100, I wrote: > Here is a patch to remove the __OFFLOAD_SYMBOL__ variable/formal > parameter, as discussed in . > > But -- I now wonder whether that's actually the issue that has been > reported i

[PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter (was: Merge current set of OpenACC changes from gomp-4_0-branch)

2015-01-16 Thread Thomas Schwinge
, as discussed in <https://gcc.gnu.org/PR64625>, I'd like to commit this patch either way, OK? commit 4409d0129118479c1cd1adbcfa96316ac4e734b0 Author: Thomas Schwinge Date: Fri Jan 16 20:12:12 2015 +0100 [PR libgomp/64625] Remove __OFFLOAD_TABLE__ variable/formal parameter.