[PATCH] D132450: StructurizeCFG: Set Undef for non-predecessors in setPhiValues()

Ruiling, Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 25 03:12:15 PDT 2022


ruiling added a comment.

In D132450#3748269 <https://reviews.llvm.org/D132450#3748269>, @fhahn wrote:

> Should this use poison instead of undef?

If I understand Poison correctly, Poison should work the same as Undef when they are used as incoming value to PHI, right? If so, changing to poison should work, but that should be in a separate change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132450



More information about the llvm-commits mailing list