commit 845a7968bce2857ae1c13ff2789ed70ece6c01a7
author Nemanja Ivanovic <nemanja.i.ibm@gmail.com> Wed Jul 05 04:51:29 2017 +0000
committer Nemanja Ivanovic <nemanja.i.ibm@gmail.com> Wed Jul 05 04:51:29 2017 +0000
tree 8eeff03747a0a4507d735a52e004fed1a0ae0dbc
parent 57495596bd9e0f911b0586be873d238c6d591a08

[PowerPC] Fix for PR33636

Remove casts to a constant when a node can be an undef.

Differential Revision: https://reviews.llvm.org/D34808

llvm-svn: 307120

llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp

diff --git a/llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp b/llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp
index 535b9de..e730564 100644
--- a/llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp
+++ b/llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp
@@ -4034,11 +4034,13 @@
                                             O0.getNode(), O1.getNode());
     };
 
+    // FIXME: When the semantics of the interaction between select and undef
+    // are clearly defined, it may turn out to be unnecessary to break here.
     SDValue TrueRes = TryFold(ConstTrue);
-    if (!TrueRes)
+    if (!TrueRes || TrueRes.isUndef())
       break;
     SDValue FalseRes = TryFold(ConstFalse);
-    if (!FalseRes)
+    if (!FalseRes || FalseRes.isUndef())
       break;
 
     // For us to materialize these using one instruction, we must be able to