On Thu, Jan 12, 2017 at 12:38 PM, Matt Turner wrote:
> On Thu, Dec 1, 2016 at 1:51 PM, Jason Ekstrand
> wrote:
> > This fixes a bug in code motion that occurred when the best block is the
> > same as the schedule early block. In this case, because we're checking
> > (lca != def->parent_instr->b
On Thu, Dec 1, 2016 at 1:51 PM, Jason Ekstrand wrote:
> This fixes a bug in code motion that occurred when the best block is the
> same as the schedule early block. In this case, because we're checking
> (lca != def->parent_instr->block) at the top of the loop, we never get to
> the check for loo
This fixes a bug in code motion that occurred when the best block is the
same as the schedule early block. In this case, because we're checking
(lca != def->parent_instr->block) at the top of the loop, we never get to
the check for loop depth so we wouldn't move it out of the loop. This
commit re