[PATCH] D83257: [SCCP] Handle assume predictes
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 6 13:47:28 PDT 2020
nikic marked an inline comment as done.
nikic added inline comments.
================
Comment at: llvm/lib/Transforms/Scalar/SCCP.cpp:1337
IV, &CB,
ValueLatticeElement::getRange(NewCR, /*MayIncludeUndef=*/true));
return;
----------------
We could set MayIncludeUndef=false for assumes (as undef/poison has always been UB there), but I didn't think it worthwhile to make the distinction, as we plan to flip this for branches in the future anyway.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83257/new/
https://reviews.llvm.org/D83257
More information about the llvm-commits
mailing list