[PATCH] D46046: [SCEV] Do not use induction in isKnownPredicate for simplification umax

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 29 11:31:13 PDT 2018


sanjoy added a comment.

I'm a bit worried about adding yet another `IsKnownViaXXX` codepath (there are already too many!).  Can we be a bit smarter in `isKnownPredicate` so that it avoids the recursive compile time blowup?  If that's not possible, can we instead push the logic in `isKnownPredicateWithoutInduction` into `isKnownViaNonRecursiveReasoning`?


https://reviews.llvm.org/D46046





More information about the llvm-commits mailing list