[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