[llvm] [SCEVExpander] Expand UDiv avoiding UB when in seq_min/max. (PR #92177)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 17 13:54:35 PDT 2024
fhahn wrote:
> LGTM, though I'd see this as just a first step of a broader change. As followups, we should:
>
> * Make SafeUDivMode the default, and disable it in LV instead.
> * Make isSafeToExpand only check for addrec expansion validity, not udiv by zero.
Sounds good to me thanks, there is definitely plenty of room for additional improvements as follow-ups!
https://github.com/llvm/llvm-project/pull/92177
More information about the llvm-commits
mailing list