[llvm] r210888 - R600: Don't call setOperationAction with things that aren't opcodes.

Matt Arsenault Matthew.Arsenault at amd.com
Fri Jun 13 00:44:39 PDT 2014

Author: arsenm
Date: Fri Jun 13 02:44:38 2014
New Revision: 210888

URL: http://llvm.org/viewvc/llvm-project?rev=210888&view=rev
R600: Don't call setOperationAction with things that aren't opcodes.

CondCode actions are set with setCondCodeAction.
This should have been a harmless bug since the values seem to only
collide only with nodes that don't need to be handled, and these are
already correctly setup elsewhere.


Modified: llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp?rev=210888&r1=210887&r2=210888&view=diff
--- llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/R600/AMDILISelLowering.cpp Fri Jun 13 02:44:38 2014
@@ -106,14 +106,6 @@ void AMDGPUTargetLowering::InitAMDILLowe
   for (MVT VT : FloatTypes) {
     // IL does not have these operations for floating point types
     setOperationAction(ISD::FP_ROUND_INREG, VT, Expand);
-    setOperationAction(ISD::SETOLT, VT, Expand);
-    setOperationAction(ISD::SETOGE, VT, Expand);
-    setOperationAction(ISD::SETOGT, VT, Expand);
-    setOperationAction(ISD::SETOLE, VT, Expand);
-    setOperationAction(ISD::SETULT, VT, Expand);
-    setOperationAction(ISD::SETUGE, VT, Expand);
-    setOperationAction(ISD::SETUGT, VT, Expand);
-    setOperationAction(ISD::SETULE, VT, Expand);
   for (MVT VT : IntTypes) {

More information about the llvm-commits mailing list