[PATCH] D65148: [SimplifyCFG] Bump phi-node-folding-threshold from 2 to 3

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


lebedev.ri added a comment.

To be noted, i'm changint this pass 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()`).
While i suppose that could be addressed via alternative means (teach some other pass that
t is okay to hoist this `@llvm.umul.with.overflow` here?),
the second motivational case is separate and remains..


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65148





More information about the llvm-commits mailing list