[PATCH] D59363: [SelectionDAG] Add icmp UNDEF handling to SelectionDAG::FoldSetCC

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 17 14:31:20 PDT 2019


uweigand added a comment.

In D59363#1431933 <https://reviews.llvm.org/D59363#1431933>, @jonpa wrote:

> I think the SystemZ test changes look ok (by replacing the undef operands with an argument the tests are more or less unaffected by this patch), but as usual I will let Uli do the formal approval.


The SystemZ test changes LGTM.  It's probably better anyway to have real arguments instead of undefs ...


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59363





More information about the llvm-commits mailing list