[PATCH] D23694: [NVPTX] Switch nvptx-use-infer-addrspace to true.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 18 14:30:26 PDT 2016


jlebar updated this revision to Diff 68608.
jlebar added a comment.

Remove out-of-date comment.


https://reviews.llvm.org/D23694

Files:
  llvm/lib/Target/NVPTX/NVPTXInferAddressSpaces.cpp
  llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp

Index: llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp
===================================================================
--- llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp
+++ llvm/lib/Target/NVPTX/NVPTXTargetMachine.cpp
@@ -51,7 +51,7 @@
 using namespace llvm;
 
 static cl::opt<bool> UseInferAddressSpaces(
-    "nvptx-use-infer-addrspace", cl::init(false), cl::Hidden,
+    "nvptx-use-infer-addrspace", cl::init(true), cl::Hidden,
     cl::desc("Optimize address spaces using NVPTXInferAddressSpaces instead of "
              "NVPTXFavorNonGenericAddrSpaces"));
 
Index: llvm/lib/Target/NVPTX/NVPTXInferAddressSpaces.cpp
===================================================================
--- llvm/lib/Target/NVPTX/NVPTXInferAddressSpaces.cpp
+++ llvm/lib/Target/NVPTX/NVPTXInferAddressSpaces.cpp
@@ -87,9 +87,6 @@
 // Finally, it fixes the undef in %y' so that
 //   %y' = phi float addrspace(3)* [ %input, %y2' ]
 //
-// TODO: This pass is experimental and not enabled by default. Users can turn it
-// on by setting the -nvptx-use-infer-addrspace flag of llc. We plan to replace
-// NVPTXNonFavorGenericAddrSpaces with this pass shortly.
 //===----------------------------------------------------------------------===//
 
 #define DEBUG_TYPE "nvptx-infer-addrspace"


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23694.68608.patch
Type: text/x-patch
Size: 1270 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160818/34987720/attachment.bin>


More information about the llvm-commits mailing list