[PATCH] D128251: [SCCP] Only handle unknown lattice values in resolvedUndefsIn()

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 21 01:45:05 PDT 2022


nikic created this revision.
nikic added reviewers: efriedma, fhahn.
Herald added a subscriber: hiraditya.
Herald added a project: All.
nikic requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

This is a minor refinement of resolvedUndefsIn(), mostly for clarity. If the value of an instruction is `undef`, then that's already a legal final result -- we can safely rauw such an instruction with `undef`. We only need to mark `unknown` values as overdefined, as that's the result we get for an instruction that has not been processed because it has an undef operand.

Depends on D126962 <https://reviews.llvm.org/D126962>.


https://reviews.llvm.org/D128251

Files:
  llvm/lib/Transforms/Utils/SCCPSolver.cpp
  llvm/test/Transforms/SCCP/PR26044.ll
  llvm/test/Transforms/SCCP/ipsccp-phi-one-pred-dead.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D128251.438596.patch
Type: text/x-patch
Size: 3969 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220621/f3e7d460/attachment.bin>


More information about the llvm-commits mailing list