Re: [PATCH] calls.c precompute_register_parameters for TLS

2020-06-22 Thread Richard Sandiford
David Edelsohn writes: > On Sun, Mar 29, 2020 at 6:44 AM Richard Sandiford > wrote: >> Well, this is all about what (symbol_ref NAME) means for a TLS symbol >> (with NAME always assumed to be TLS below). For all GCC ports it means >> the actual address of NAME for the current thread. AIUI that

Re: [PATCH] calls.c precompute_register_parameters for TLS

2020-06-22 Thread David Edelsohn via Gcc-patches
On Sun, Mar 29, 2020 at 6:44 AM Richard Sandiford wrote: > > David Edelsohn writes: > > On Sat, Mar 28, 2020 at 6:42 AM Richard Sandiford > > wrote: > >> > >> David Edelsohn via Gcc-patches writes: > >> > This patch is for an AIX problem, but the only robust solution is in > >> > common code:

Re: [PATCH] calls.c precompute_register_parameters for TLS

2020-03-29 Thread Richard Sandiford
David Edelsohn writes: > On Sat, Mar 28, 2020 at 6:42 AM Richard Sandiford > wrote: >> >> David Edelsohn via Gcc-patches writes: >> > This patch is for an AIX problem, but the only robust solution is in >> > common code: calls.c:precompute_register_parameters(). >> > >> > AIX, like other

Re: [PATCH] calls.c precompute_register_parameters for TLS

2020-03-28 Thread David Edelsohn via Gcc-patches
On Sat, Mar 28, 2020 at 6:42 AM Richard Sandiford wrote: > > David Edelsohn via Gcc-patches writes: > > This patch is for an AIX problem, but the only robust solution is in > > common code: calls.c:precompute_register_parameters(). > > > > AIX, like other platforms, needs to call a function to

Re: [PATCH] calls.c precompute_register_parameters for TLS

2020-03-28 Thread Richard Sandiford
David Edelsohn via Gcc-patches writes: > This patch is for an AIX problem, but the only robust solution is in > common code: calls.c:precompute_register_parameters(). > > AIX, like other platforms, needs to call a function to obtain the > pointer to thread-local storage. If the thread local

Re: [PATCH] calls.c precompute_register_parameters for TLS

2020-03-28 Thread Eric Botcazou
> I'm forced to fall back to a targeted patch in > calls.c:precompute_register_parameters() that tests exactly for TLS > symbols on AIX instead of using the legitimate_constant_p hook to test > for TLS symbols. > > I could create a new target hook for this specific use in calls.c and > define it

[PATCH] calls.c precompute_register_parameters for TLS

2020-03-27 Thread David Edelsohn via Gcc-patches
This patch is for an AIX problem, but the only robust solution is in common code: calls.c:precompute_register_parameters(). AIX, like other platforms, needs to call a function to obtain the pointer to thread-local storage. If the thread local variable is referenced as a parameter to a function