[llvm] r304221 - [SelectionDAG] Remove special case for ISD::FPOWI from the strict FP intrinsic handling.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Tue May 30 10:12:18 PDT 2017


Author: ctopper
Date: Tue May 30 12:12:18 2017
New Revision: 304221

URL: http://llvm.org/viewvc/llvm-project?rev=304221&view=rev
Log:
[SelectionDAG] Remove special case for ISD::FPOWI from the strict FP intrinsic handling.

This code was compensating for FPOWI defaulting to Legal and many targets not changing it to Expand. This was fixed in r304215 to default to Expand so this special handling should no longer be necessary.

Modified:
    llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp

Modified: llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp?rev=304221&r1=304220&r2=304221&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp Tue May 30 12:12:18 2017
@@ -925,10 +925,6 @@ getStrictFPOpcodeAction(const TargetLowe
   if (Action != TargetLowering::Legal)
     Action = TargetLowering::Expand;
 
-  // ISD::FPOWI returns 'Legal' even though it should be expanded.
-  if (Opcode == ISD::STRICT_FPOWI && Action == TargetLowering::Legal)
-    Action = TargetLowering::Expand;
-
   return Action;
 }
 




More information about the llvm-commits mailing list