[PATCH] D104636: [LoopIdiom] [LoopNest] let the pass deal with runtime memset size

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Yes, I guess you need to version to handle that case in particular.  I'd like to avoid versioning in simpler cases, though.

I have to ask; is that case actually showing up in real code?  Usually, people prefer for loops over do-while loops in situations like that.  If you write an equivalent test using for loops, SCEV should simplify the backedge-taken count.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104636/new/

https://reviews.llvm.org/D104636