[PATCH] D97495: [SimplifyCFG] avoid illegal phi with both poison and undef

Juneyoung Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 26 00:16:28 PST 2021


aqjune added a comment.

What about updating redirectValuesFromPredecessorsToPhi to lower conflicting poison to undef?
We can maintain two sets of incoming BBs, say BBundef and BBpoison that have poison incoming values and undef incoming values.
At the end of redirectValuesFromPredecessorsToPhi, if BBundef and BBpoison overlaps, there are operands that need to be updated.
It will keep the transformation enabled.


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

https://reviews.llvm.org/D97495



More information about the llvm-commits mailing list