Re: [PATCH] rs6000: Make CSE'ing __tls_get_addr calls possible

2019-08-31 Thread Segher Boessenkool
On Sat, Aug 31, 2019 at 02:17:30PM -0500, Segher Boessenkool wrote: > Hi all, > > On Sat, Mar 23, 2019 at 11:46:12PM +, Segher Boessenkool wrote: > > CSE does not consider calls, not even const calls. This patch puts a > > REG_EQUAL note on the pseudo we assign the __tls_get_addr result to,

Re: [PATCH] rs6000: Make CSE'ing __tls_get_addr calls possible

2019-08-31 Thread Segher Boessenkool
Hi all, On Sat, Mar 23, 2019 at 11:46:12PM +, Segher Boessenkool wrote: > CSE does not consider calls, not even const calls. This patch puts a > REG_EQUAL note on the pseudo we assign the __tls_get_addr result to, > so that those pseudos can be CSE'd and the extra calls deleted as dead >

Re: [PATCH] rs6000: Make CSE'ing __tls_get_addr calls possible

2019-03-24 Thread Eric Botcazou
> There were REG_EQUAL notes on these tls calls in the past, but I > recall removing them for one reason or another. So watch out for > fall-out from this patch! ;-) Others ports have them though, for example i386, ARM and SPARC. -- Eric Botcazou

Re: [PATCH] rs6000: Make CSE'ing __tls_get_addr calls possible

2019-03-24 Thread Segher Boessenkool
Hi Steven, On Sun, Mar 24, 2019 at 11:02:20AM +0100, Steven Bosscher wrote: > On Sun, Mar 24, 2019 at 12:46 AM Segher Boessenkool wrote: > > > > CSE does not consider calls, not even const calls. This patch puts a > > REG_EQUAL note on the pseudo we assign the __tls_get_addr result to, > > so

Re: [PATCH] rs6000: Make CSE'ing __tls_get_addr calls possible

2019-03-24 Thread Steven Bosscher
On Sun, Mar 24, 2019 at 12:46 AM Segher Boessenkool wrote: > > CSE does not consider calls, not even const calls. This patch puts a > REG_EQUAL note on the pseudo we assign the __tls_get_addr result to, > so that those pseudos can be CSE'd and the extra calls deleted as dead > code. Hi Segher,

[PATCH] rs6000: Make CSE'ing __tls_get_addr calls possible

2019-03-23 Thread Segher Boessenkool
CSE does not consider calls, not even const calls. This patch puts a REG_EQUAL note on the pseudo we assign the __tls_get_addr result to, so that those pseudos can be CSE'd and the extra calls deleted as dead code. CSE should really handle const calls directly, but it is stage 4. Tested on