Hello Richard,
> This patch adjusts the dump output from
modulo-sched.c:get_schdedule_window.
> Dump output is very much down to personal preference, so please feel free
> just to reject the change.
The output format looks great to me although I cannot approve it.
Thanks,
Revital
Hello,
> On Mon, Jan 2, 2012 at 3:30 PM, Richard Sandiford
> wrote:
> > Ayal Zaks writes:
> >> + for (i = 0; i < ira_pressure_classes_num; i++)
> >> + {
> >> + enum reg_class pressure_class;
> >> +
> >> + pressure_class = ira_pressure_classes[i];
> >> +
> >> + if (max_reg_pre
Hello,
The patch below fixes ICE reported in PR51794.
It avoids creating DDG edges for register uses of class DF_REF_ARTIFICIAL
as
the latter does not have real instructions for them and thus calling
BLOCK_FOR_INSN fails.
Tested and bootstrap on ppc64-redhat-linux, enabling SMS on loops with SC
Hi Roman,
> While building a data dependency graph for loop a ddg edge for some pair
> of instructions with inter-loop dependency should be created only if
> there is no edge for intra-loop dependency between these instructions.
> Creating both of edges leads sometimes to the fact that function
I
Hello Roman,
> This patch should be applied only after pending patches by Revital. This
patch
> significantly enhances the existing implementation of the SMS. Patch
adds
> support of scheduling loops without doloop pattern. The loop should meet
the
> following requirements.
Thanks for the patch
Hi,
Thanks for the review!
>
> >Changelog:
> >
> > (sms_schedule_by_order): Update call to get_sched_window.
> > all set_must_precede_follow.
> ^^^
> call
Done.
> >+/* Set bitmaps TMP_FOLLOW and TMP_PRECEDE to MUST_FOLLOW and
MUST_PRECEDE
> >+ respectively only if
Hello Alexandre
> I think this will restore proper functioning to SMS in the presence of
> debug insns. A while ago, we'd never generate deps of non-debug insns
> on debug insns. I introduced them to enable sched to adjust (reset)
> debug insns when non-debug insns were moved before them. I bel
Hello,
(sorry for multiple copies of this email)
This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file. It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloo
Hello,
The attached patch includes enhancements for SMS to support targets
that their doloop part is not decoupled from the rest of the loop's
instructions, as SMS currently requires. (ARM is an example for such
target, where the loop's instructions might use CC which is used in the
doloop part)
