[llvm-commits] [llvm] r110226 - /llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp

Dale Johannesen dalej at apple.com
Wed Aug 4 11:07:17 PDT 2010


Author: johannes
Date: Wed Aug  4 13:07:17 2010
New Revision: 110226

URL: http://llvm.org/viewvc/llvm-project?rev=110226&view=rev
Log:
Remove switch for disabling ARM tail calls.  They
seem to be working correctly.  No functional change.


Modified:
    llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp

Modified: llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp?rev=110226&r1=110225&r2=110226&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMISelLowering.cpp Wed Aug  4 13:07:17 2010
@@ -51,12 +51,6 @@
 
 STATISTIC(NumTailCalls, "Number of tail calls");
 
-// This option should go away when tail calls fully work.
-static cl::opt<bool>
-EnableARMTailCalls("arm-tail-calls", cl::Hidden,
-  cl::desc("Generate tail calls (TEMPORARY OPTION)."),
-  cl::init(true));
-
 // This option should go away when Machine LICM is smart enough to hoist a 
 // reg-to-reg VDUP.
 static cl::opt<bool>
@@ -1123,9 +1117,6 @@
   MachineFunction &MF = DAG.getMachineFunction();
   bool IsStructRet    = (Outs.empty()) ? false : Outs[0].Flags.isSRet();
   bool IsSibCall = false;
-  // Temporarily disable tail calls so things don't break.
-  if (!EnableARMTailCalls)
-    isTailCall = false;
   if (isTailCall) {
     // Check if it's really possible to do a tail call.
     isTailCall = IsEligibleForTailCallOptimization(Callee, CallConv,





More information about the llvm-commits mailing list