On 08/27/2014 09:32 AM, David Malcolm wrote:
> * gcc/config/mn10300/mn10300.c (is_load_insn): Rename to...
> (set_is_load_p): ...this, updating to work on a SET pattern rather
> than an insn.
> (is_store_insn): Rename to...
> (set_is_store_p): ...this, updating to work
On Wed, 2014-08-27 at 09:11 -0700, Richard Henderson wrote:
> On 08/27/2014 08:48 AM, David Malcolm wrote:
> > Alternatively, should this simply use "single_set"?
>
> Yes.
>
> > (though I think that's a more invasive change, especially since some of
> > the logic is for non-SETs).
>
> I don't th
On 08/27/2014 08:48 AM, David Malcolm wrote:
> Alternatively, should this simply use "single_set"?
Yes.
> (though I think that's a more invasive change, especially since some of
> the logic is for non-SETs).
I don't think that's the case. Take the tests in order:
if (mn10300_tune_cpu == PROC
On Tue, 2014-08-19 at 11:02 -0700, Richard Henderson wrote:
> On 08/06/2014 10:19 AM, David Malcolm wrote:
> > @@ -2772,11 +2772,11 @@ mn10300_adjust_sched_cost (rtx insn, rtx link, rtx
> > dep, int cost)
> >if (!TARGET_AM33)
> > return 1;
> >
> > - if (GET_CODE (insn) == PARALLEL)
> >
On 08/06/2014 10:19 AM, David Malcolm wrote:
> @@ -2772,11 +2772,11 @@ mn10300_adjust_sched_cost (rtx insn, rtx link, rtx
> dep, int cost)
>if (!TARGET_AM33)
> return 1;
>
> - if (GET_CODE (insn) == PARALLEL)
> -insn = XVECEXP (insn, 0, 0);
> + if (GET_CODE (PATTERN (insn)) == PAR
On 08/06/14 11:19, David Malcolm wrote:
gcc/
* config/mn10300/mn10300.c (mn10300_adjust_sched_cost): Fix the
handling of PARALLEL to work on PATTERN (insn) and PATTERN (dep),
rather than just on insn, dep themselves. The latter are insns,
and thus can't be PARALLE
