<div dir="ltr">Hrm.  This is probably okay temporarily, but I've been meaning to commit MC support (so InstPrinter would actually be useful).  I'll leave it up to you.  If you want to commit this to get rid of the warnings, that's fine.  If not, the warnings should go away soon anyway.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 1, 2013 at 6:54 AM, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi jholewinski,<br>
<br>
When doing a clean build, I get messages from ranlib about the NVPTXAsmPrinter library containing no symbols. It appears to be completely empty, with its functionality subsumed into the CodeGen library (NVPTXAsmPrinter.cpp specifically).<br>

<br>
The attached patch removes these files. Ok to commit, NVPTX people?<br>
<br>
Cheers.<br>
<br>
Tim.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1255" target="_blank">http://llvm-reviews.chandlerc.com/D1255</a><br>
<br>
Files:<br>
  lib/Target/NVPTX/CMakeLists.txt<br>
  lib/Target/NVPTX/InstPrinter/CMakeLists.txt<br>
  lib/Target/NVPTX/InstPrinter/LLVMBuild.txt<br>
  lib/Target/NVPTX/InstPrinter/Makefile<br>
  lib/Target/NVPTX/InstPrinter/NVPTXInstPrinter.cpp<br>
  lib/Target/NVPTX/LLVMBuild.txt<br>
  lib/Target/NVPTX/MCTargetDesc/LLVMBuild.txt<br>
  lib/Target/NVPTX/Makefile<br>
<br>
Index: lib/Target/NVPTX/CMakeLists.txt<br>
===================================================================<br>
--- lib/Target/NVPTX/CMakeLists.txt<br>
+++ lib/Target/NVPTX/CMakeLists.txt<br>
@@ -32,5 +32,4 @@<br>
 add_dependencies(LLVMNVPTXCodeGen intrinsics_gen)<br>
<br>
 add_subdirectory(TargetInfo)<br>
-add_subdirectory(InstPrinter)<br>
 add_subdirectory(MCTargetDesc)<br>
Index: lib/Target/NVPTX/InstPrinter/CMakeLists.txt<br>
===================================================================<br>
--- lib/Target/NVPTX/InstPrinter/CMakeLists.txt<br>
+++ /dev/null<br>
@@ -1,7 +0,0 @@<br>
-include_directories( ${CMAKE_CURRENT_BINARY_DIR}/.. ${CMAKE_CURRENT_SOURCE_DIR}/.. )<br>
-<br>
-add_llvm_library(LLVMNVPTXAsmPrinter<br>
-  NVPTXInstPrinter.cpp<br>
-  )<br>
-<br>
-add_dependencies(LLVMNVPTXAsmPrinter NVPTXCommonTableGen)<br>
Index: lib/Target/NVPTX/InstPrinter/LLVMBuild.txt<br>
===================================================================<br>
--- lib/Target/NVPTX/InstPrinter/LLVMBuild.txt<br>
+++ /dev/null<br>
@@ -1,23 +0,0 @@<br>
-;===- ./lib/Target/NVPTX/InstPrinter/LLVMBuild.txt -------------*- Conf -*--===;<br>
-;<br>
-;                     The LLVM Compiler Infrastructure<br>
-;<br>
-; This file is distributed under the University of Illinois Open Source<br>
-; License. See LICENSE.TXT for details.<br>
-;<br>
-;===------------------------------------------------------------------------===;<br>
-;<br>
-; This is an LLVMBuild description file for the components in this subdirectory.<br>
-;<br>
-; For more information on the LLVMBuild system, please see:<br>
-;<br>
-;   <a href="http://llvm.org/docs/LLVMBuild.html" target="_blank">http://llvm.org/docs/LLVMBuild.html</a><br>
-;<br>
-;===------------------------------------------------------------------------===;<br>
-<br>
-[component_0]<br>
-type = Library<br>
-name = NVPTXAsmPrinter<br>
-parent = NVPTX<br>
-required_libraries = MC Support<br>
-add_to_library_groups = NVPTX<br>
Index: lib/Target/NVPTX/InstPrinter/Makefile<br>
===================================================================<br>
--- lib/Target/NVPTX/InstPrinter/Makefile<br>
+++ /dev/null<br>
@@ -1,15 +0,0 @@<br>
-##===- lib/Target/NVPTX/AsmPrinter/Makefile ----------------*- Makefile -*-===##<br>
-#<br>
-#                                                                                      The LLVM Compiler Infrastructure<br>
-#<br>
-# This file is distributed under the University of Illinois Open Source<br>
-# License. See LICENSE.TXT for details.<br>
-#<br>
-##===----------------------------------------------------------------------===##<br>
-LEVEL = ../../../..<br>
-LIBRARYNAME = LLVMNVPTXAsmPrinter<br>
-<br>
-# Hack: we need to include 'main' ptx target directory to grab private headers<br>
-CPP.Flags += -I$(PROJ_OBJ_DIR)/.. -I$(PROJ_SRC_DIR)/..<br>
-<br>
-include $(LEVEL)/Makefile.common<br>
Index: lib/Target/NVPTX/InstPrinter/NVPTXInstPrinter.cpp<br>
===================================================================<br>
--- lib/Target/NVPTX/InstPrinter/NVPTXInstPrinter.cpp<br>
+++ /dev/null<br>
@@ -1 +0,0 @@<br>
-// Placeholder<br>
Index: lib/Target/NVPTX/LLVMBuild.txt<br>
===================================================================<br>
--- lib/Target/NVPTX/LLVMBuild.txt<br>
+++ lib/Target/NVPTX/LLVMBuild.txt<br>
@@ -16,7 +16,7 @@<br>
 ;===------------------------------------------------------------------------===;<br>
<br>
 [common]<br>
-subdirectories = InstPrinter MCTargetDesc TargetInfo<br>
+subdirectories = MCTargetDesc TargetInfo<br>
<br>
 [component_0]<br>
 type = TargetGroup<br>
Index: lib/Target/NVPTX/MCTargetDesc/LLVMBuild.txt<br>
===================================================================<br>
--- lib/Target/NVPTX/MCTargetDesc/LLVMBuild.txt<br>
+++ lib/Target/NVPTX/MCTargetDesc/LLVMBuild.txt<br>
@@ -19,5 +19,5 @@<br>
 type = Library<br>
 name = NVPTXDesc<br>
 parent = NVPTX<br>
-required_libraries = MC NVPTXAsmPrinter NVPTXInfo Support<br>
+required_libraries = MC NVPTXInfo Support<br>
 add_to_library_groups = NVPTX<br>
Index: lib/Target/NVPTX/Makefile<br>
===================================================================<br>
--- lib/Target/NVPTX/Makefile<br>
+++ lib/Target/NVPTX/Makefile<br>
@@ -18,6 +18,6 @@<br>
                NVPTXGenRegisterInfo.inc \<br>
                NVPTXGenSubtargetInfo.inc<br>
<br>
-DIRS = InstPrinter TargetInfo MCTargetDesc<br>
+DIRS = TargetInfo MCTargetDesc<br>
<br>
 include $(LEVEL)/Makefile.common<br>
</blockquote></div><br><br clear="all"><br>-- <br><br><div>Thanks,</div><div><br></div><div>Justin Holewinski</div>
</div>