[PATCH] Fix optimisations of SELECT_CC which assumed result is boolean

Matt Arsenault Matthew.Arsenault at amd.com
Fri Nov 14 14:57:30 PST 2014


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:648-650
@@ -647,1 +647,5 @@
 
+  if (TLI.getBooleanContents(N.getValueType()) ==
+      TargetLowering::UndefinedBooleanContent)
+    return false;
+
----------------
Is this really necessary? I believe isConstTrue should do the correct thing for for UndefinedBooleanContent

http://reviews.llvm.org/D6249






More information about the llvm-commits mailing list