[PATCH, SMS] Fix PR51794

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

Re: [PATCH SMS 2/2, RFC] Register pressure estimation for the partial schedule (re-submission)

Hello, On Mon, Jan 2, 2012 at 3:30 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Ayal Zaks ayal.z...@gmail.com writes: +  for (i = 0; i ira_pressure_classes_num; i++) +    { +      enum reg_class pressure_class; + +      pressure_class = ira_pressure_classes[i]; + +  

Re: Adjust debug output from SMS's get_schdedule_window

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

Re: Patches ping

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 cycle C falls

Re: [PATCH 3/9] [SMS] Eliminate redundant edges

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 It

Re: [PATCH 5/9] [SMS] Support new loop pattern

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! I

[PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​​p.

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

Re: debug insns in SMS (was: Re: Debug_insn)

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

[PATCH, SMS] Support closing_branch_deps

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)