[PATCH] D111493: [IVUsers] Check for preheader instead of loop simplify form

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 11 08:48:39 PDT 2021


reames added a comment.

LGTM from me as well.

Glancing through the SCEVExpander code, I only see one unconditional use of a preheader assumption, and that looks easy to bail out of.  Are you sure we need this restriction at all?  (Landing this and then investigating is encouraged.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111493



More information about the llvm-commits mailing list