forked from OSchip/llvm-project
a99d420a93
This is split from D105216, it handles only a subset of the cases in that patch. Specifically, the issue being fixed is that the code incorrectly assumed that (Start-Stide) < End implied that the backedge was taken at least once. This is not true when e.g. Start = 4, Stride = 2, and End = 3. Note that we often do produce the right backedge taken count despite the flawed reasoning. The fix chosen here is to use an alternate form of uceil (ceiling of unsigned divide) lowering which is safe when max(RHS,Start) > Start - Stride. (Note that signedness of both max expression and comparison depend on the signedness of the comparison being analyzed, and that overflow in the Start - Stride expression is allowed.) Note that this is weaker than proving the backedge is taken because it allows start - stride < end < start. Some cases which can't be proven safe are sent down the generic path, and we do end up generating less optimal expressions in a few cases. Credit for coming up with the approach goes entirely to Eli. I just split it off, tweaked the comments a bit, and did some additional testing. Differential Revision: https://reviews.llvm.org/D105942 |
||
|---|---|---|
| .. | ||
| basic.ll | ||
| basic32iters.ll | ||
| complex_reroll.ll | ||
| external_use.ll | ||
| extra_instr.ll | ||
| indvar_with_ext.ll | ||
| negative.ll | ||
| nonconst_lb.ll | ||
| ptrindvar.ll | ||
| reduction.ll | ||
| reroll_with_dbg.ll | ||