Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-06 Thread Richard Earnshaw (lists)
On 06/11/2018 20:18, Segher Boessenkool wrote: > On Tue, Nov 06, 2018 at 07:43:36PM +, Richard Earnshaw (lists) wrote: >> On 06/11/2018 18:18, Segher Boessenkool wrote: >>> On Tue, Nov 06, 2018 at 11:46:53AM +, Richard Earnshaw (lists) wrote: Well it generates new 'light-weight' prolog

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-06 Thread Segher Boessenkool
On Tue, Nov 06, 2018 at 07:43:36PM +, Richard Earnshaw (lists) wrote: > On 06/11/2018 18:18, Segher Boessenkool wrote: > > On Tue, Nov 06, 2018 at 11:46:53AM +, Richard Earnshaw (lists) wrote: > >> Well it generates new 'light-weight' prologue and epilogue sequences for > >> the 'shrunk' co

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-06 Thread Richard Earnshaw (lists)
On 06/11/2018 19:43, Richard Earnshaw (lists) wrote: > On 06/11/2018 18:18, Segher Boessenkool wrote: >> Hi Richard, >> >> On Tue, Nov 06, 2018 at 11:46:53AM +, Richard Earnshaw (lists) wrote: >>> On 06/11/2018 01:40, Segher Boessenkool wrote: Hi Richard, On Mon, Nov 05, 2018 at

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-06 Thread Richard Earnshaw (lists)
On 06/11/2018 18:18, Segher Boessenkool wrote: > Hi Richard, > > On Tue, Nov 06, 2018 at 11:46:53AM +, Richard Earnshaw (lists) wrote: >> On 06/11/2018 01:40, Segher Boessenkool wrote: >>> Hi Richard, >>> >>> On Mon, Nov 05, 2018 at 10:09:30AM +, Richard Earnshaw (lists) wrote: >>> Sho

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-06 Thread Segher Boessenkool
Hi Richard, On Tue, Nov 06, 2018 at 11:46:53AM +, Richard Earnshaw (lists) wrote: > On 06/11/2018 01:40, Segher Boessenkool wrote: > > Hi Richard, > > > > On Mon, Nov 05, 2018 at 10:09:30AM +, Richard Earnshaw (lists) wrote: > > Shouldn't you be able to do this per function at least?

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-06 Thread Richard Earnshaw (lists)
On 06/11/2018 01:40, Segher Boessenkool wrote: > Hi Richard, > > On Mon, Nov 05, 2018 at 10:09:30AM +, Richard Earnshaw (lists) wrote: > Shouldn't you be able to do this per function at least? do what per function? track speculation? >>> >>> disable shrink-wrapping only when any

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-05 Thread Segher Boessenkool
Hi Richard, On Mon, Nov 05, 2018 at 10:09:30AM +, Richard Earnshaw (lists) wrote: > >>> Shouldn't you be able to do this per function at least? > >> > >> do what per function? track speculation? > > > > disable shrink-wrapping only when any speculation was there > > (this is about __bultin_s

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-05 Thread Richard Earnshaw (lists)
On 05/11/2018 10:05, Richard Biener wrote: > On Fri, Nov 2, 2018 at 3:04 PM Richard Earnshaw (lists) > wrote: >> >> On 02/11/2018 13:53, Richard Biener wrote: >>> On Fri, Nov 2, 2018 at 2:38 PM Richard Earnshaw (lists) >>> wrote: Although there's no fundamental reason why shrink wrappin

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-05 Thread Richard Biener
On Fri, Nov 2, 2018 at 3:04 PM Richard Earnshaw (lists) wrote: > > On 02/11/2018 13:53, Richard Biener wrote: > > On Fri, Nov 2, 2018 at 2:38 PM Richard Earnshaw (lists) > > wrote: > >> > >> Although there's no fundamental reason why shrink wrapping and > >> speculation tracking are incompatible,

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-02 Thread Richard Earnshaw (lists)
On 02/11/2018 13:53, Richard Biener wrote: > On Fri, Nov 2, 2018 at 2:38 PM Richard Earnshaw (lists) > wrote: >> >> Although there's no fundamental reason why shrink wrapping and >> speculation tracking are incompatible, a phase-ordering requirement (we >> need to do speculation tracking before th

Re: [aarch64] disable shrink wrapping when tracking speculative execution

2018-11-02 Thread Richard Biener
On Fri, Nov 2, 2018 at 2:38 PM Richard Earnshaw (lists) wrote: > > Although there's no fundamental reason why shrink wrapping and > speculation tracking are incompatible, a phase-ordering requirement (we > need to do speculation tracking before the final basic block clean-up) > means that the shri

[aarch64] disable shrink wrapping when tracking speculative execution

2018-11-02 Thread Richard Earnshaw (lists)
Although there's no fundamental reason why shrink wrapping and speculation tracking are incompatible, a phase-ordering requirement (we need to do speculation tracking before the final basic block clean-up) means that the shrink wrapping pass can undo some of the changes the speculation tracking pas