[PATCH] D79720: [IndVarSimplify][LoopUtils] Track rewrite cost per unique BB (PR45835)

dmajor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 11 09:39:55 PDT 2020


dmajor added a comment.

Note: the test case does not currently fail on trunk before this change (although it does fail on the 10.0 release branch, where I would eventually like to get this merged) because after D73744 <https://reviews.llvm.org/D73744> it is no longer blanket assumed that SCEVMinMaxExpr is high cost. Any pointers on how I could modify the test to get around that? (I'm not familiar with why this IR is considered to have a SCEVMinMaxExpr or what it would take to make it high cost.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79720





More information about the llvm-commits mailing list