[PATCH] D65147: [SimplifyCFG] FoldTwoEntryPHINode(): don't bailout on i1 PHI's if we can hoist a 'not' from incoming values

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 23 08:48:15 PDT 2019


lebedev.ri added a comment.

To be noted, i'm fixing this function in particular because it is the pass that folds the case witout `not`, in `-O3`
(`llvm/test/Transforms/PhaseOrdering/unsigned-multiply-overflow-check.ll`, `will_not_overflow()`).
I suppose the alternative approach would be to teach some other pass that it is okay to hoist this `@llvm.umul.with.overflow` here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65147





More information about the llvm-commits mailing list