RE: FW: [PATCH] Target hook for disabling the delay slot filler.

2015-10-26 Thread Simon Dardis
> On 10/23/2015 11:31 AM, Bernd Schmidt wrote: > > On 10/23/2015 04:57 PM, Simon Dardis wrote: > > > >> Patch below. Target hook renamed to > >> TARGET_NO_SPECULATION_IN_DELAY_SLOTS_P. > >> > >> Tested on mips-img-elf, no new regressions. > > > > As far as I'm concerned this is ok, and IIUC Jeff wa

Re: FW: [PATCH] Target hook for disabling the delay slot filler.

2015-10-23 Thread Jeff Law
On 10/23/2015 11:31 AM, Bernd Schmidt wrote: On 10/23/2015 04:57 PM, Simon Dardis wrote: Patch below. Target hook renamed to TARGET_NO_SPECULATION_IN_DELAY_SLOTS_P. Tested on mips-img-elf, no new regressions. As far as I'm concerned this is ok, and IIUC Jeff was on board too. This is assumin

Re: FW: [PATCH] Target hook for disabling the delay slot filler.

2015-10-23 Thread Bernd Schmidt
On 10/23/2015 04:57 PM, Simon Dardis wrote: Patch below. Target hook renamed to TARGET_NO_SPECULATION_IN_DELAY_SLOTS_P. Tested on mips-img-elf, no new regressions. As far as I'm concerned this is ok, and IIUC Jeff was on board too. This is assuming the test included a bootstrap, otherwise pl

RE: FW: [PATCH] Target hook for disabling the delay slot filler.

2015-10-23 Thread Simon Dardis
> -Original Message- > From: Jeff Law [mailto:l...@redhat.com] > Sent: 08 October 2015 20:44 > To: Simon Dardis; Bernd Schmidt > Cc: gcc-patches@gcc.gnu.org > Subject: Re: FW: [PATCH] Target hook for disabling the delay slot filler. > > On 09/18/2015 05:1

Re: FW: [PATCH] Target hook for disabling the delay slot filler.

2015-10-08 Thread Jeff Law
On 09/18/2015 05:10 AM, Simon Dardis wrote: Are you trying to say that you have the option as to what kind of branch to use? ie, "ordinary", presumably without a delay slot or one with a delay slot? Is the "ordinary" actually just a nullified delay slot or some form of likely/not likely stati

FW: [PATCH] Target hook for disabling the delay slot filler.

2015-09-18 Thread Simon Dardis
ber 2015 17:55 To: Simon Dardis; Bernd Schmidt Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] Target hook for disabling the delay slot filler. On 09/17/2015 03:52 AM, Simon Dardis wrote: > The profitability of using an ordinary branch over a delay slot branch > depends on how the delay slot

Re: [PATCH] Target hook for disabling the delay slot filler.

2015-09-17 Thread Jeff Law
On 09/17/2015 03:52 AM, Simon Dardis wrote: The profitability of using an ordinary branch over a delay slot branch depends on how the delay slot is filled. If a delay slot can be filled from an instruction preceding the branch or instructions proceeding that must be executed on both sides then it

Re: [PATCH] Target hook for disabling the delay slot filler.

2015-09-17 Thread Bernd Schmidt
On 09/17/2015 11:52 AM, Simon Dardis wrote: The profitability of using an ordinary branch over a delay slot branch depends on how the delay slot is filled. If a delay slot can be filled from an instruction preceding the branch or instructions proceeding that must be executed on both sides then it

RE: [PATCH] Target hook for disabling the delay slot filler.

2015-09-17 Thread Simon Dardis
@gcc.gnu.org Subject: Re: [PATCH] Target hook for disabling the delay slot filler. On 09/15/2015 08:27 AM, Bernd Schmidt wrote: > On 09/15/2015 04:19 PM, Simon Dardis wrote: >> This patch adds a target hook for disabling the eager delay slot >> filler which when disabled can give bett

Re: [PATCH] Target hook for disabling the delay slot filler.

2015-09-15 Thread Jeff Law
On 09/15/2015 08:27 AM, Bernd Schmidt wrote: On 09/15/2015 04:19 PM, Simon Dardis wrote: This patch adds a target hook for disabling the eager delay slot filler which when disabled can give better code. No new regressions. Ok to commit? Hmm. Whether a branch was filled by the simple or eager f

Re: [PATCH] Target hook for disabling the delay slot filler.

2015-09-15 Thread Bernd Schmidt
On 09/15/2015 04:19 PM, Simon Dardis wrote: This patch adds a target hook for disabling the eager delay slot filler which when disabled can give better code. No new regressions. Ok to commit? Hmm. Whether a branch was filled by the simple or eager filler is an implementation detail - is there

[PATCH] Target hook for disabling the delay slot filler.

2015-09-15 Thread Simon Dardis
Hello all, This patch adds a target hook for disabling the eager delay slot filler which when disabled can give better code. No new regressions. Ok to commit? Thanks, Simon gcc/ * target.def (use_eager_delay_filler_p): New hook for selectively disabling eager delay slot filler.