[llvm] r331088 - [FastISel] Actually enable local value sinking by default

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 27 14:51:25 PDT 2018


Author: rnk
Date: Fri Apr 27 14:51:25 2018
New Revision: 331088

URL: http://llvm.org/viewvc/llvm-project?rev=331088&view=rev
Log:
[FastISel] Actually enable local value sinking by default

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

Modified: llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp?rev=331088&r1=331087&r2=331088&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/FastISel.cpp Fri Apr 27 14:51:25 2018
@@ -113,11 +113,9 @@ using namespace llvm;
 
 #define DEBUG_TYPE "isel"
 
-// FIXME: Remove this when compile time issues are addressed. Do this by only
-// numbering instructions between local value map flush points instead of the
-// entire BB.
+// FIXME: Remove this after the feature has proven reliable.
 static cl::opt<bool> SinkLocalValues("fast-isel-sink-local-values",
-                                     cl::init(false), cl::Hidden,
+                                     cl::init(true), cl::Hidden,
                                      cl::desc("Sink local values in FastISel"));
 
 STATISTIC(NumFastIselSuccessIndependent, "Number of insts selected by "




More information about the llvm-commits mailing list