[PATCH] D106041: [WIP][SimpleLoopUnswitch] Re-fix introduction of UB when hoisted condition may be undef or poison

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 3 06:49:28 PDT 2021


lebedev.ri added a comment.

Much like with pushing freeze within simplifycfg, doing that in loopunswitch seems misplaced.
It's impossible to tell without a phase-ordering test (one with `-O3`, not just a single pass),
but perhaps we simply need to run/move instcombine invocation somewhere in the pipeline?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106041



More information about the llvm-commits mailing list