[PATCH] D116766: [SCEV] Poison-safe `UMin` expression

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 7 00:02:40 PST 2022


nikic added a comment.

Worth noting that we should be using "safe umin" not just for the logical or case, but also when combining exit counts from multiple exits, which is the more common case. It's probably best to do that separately, but that would give a clearer picture of how bad this is in terms of practical impact, because we have a lot more tests for that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116766



More information about the llvm-commits mailing list