For PR466:
authorReid Spencer <rspencer@reidspencer.com>
Thu, 2 Dec 2004 09:28:21 +0000 (09:28 +0000)
committerReid Spencer <rspencer@reidspencer.com>
Thu, 2 Dec 2004 09:28:21 +0000 (09:28 +0000)
Change construction of bytecode libraries from producing a single bytecode
file to producing a library containing bytecode files. This gets around the
problem of multiple symbol definitions in the linker if something like
-lc -lc is attempted on the command line. Previously this happened because
the linker would find libc.bc as a "library". It will now find libc.a which
it can simply search for missing symbols instead of linking in wholesale.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@18425 91177308-0d34-0410-b5e6-96231b3b80d8

Makefile.rules

index 9897fc87846f001bd1d15b7a16b22de0253edcfd..fc6598bb672211342aeb28ac2de160aee2036bc5 100644 (file)
@@ -232,8 +232,8 @@ ifdef TOOL_VERBOSE
   C.Flags += -v
   CXX.Flags += -v
   LD.Flags += -v
-  BCLinkLib.Flags += -v
   VERBOSE := 1
+else
 endif
 
 # Adjust settings for verbose mode
@@ -286,12 +286,11 @@ Link          = $(LIBTOOL) $(LibTool.Flags) --mode=link $(CXX) $(CPP.Flags) \
                $(CompileCommonOpts) $(LD.Flags) $(Strip)
 Relink        = $(LIBTOOL) $(LibTool.Flags) --mode=link $(CXX) $(CPP.Flags) \
                 $(CompileCommonOpts)
-BCLinkLib     = $(LLVMGCC) -shared -nostdlib $(BCLinkLib.Flags)
 LTInstall     = $(LIBTOOL) $(LibTool.Flags) --mode=install $(INSTALL)
 Burg          = $(BURG) -I $(BUILD_SRC_DIR)
 TableGen      = $(TBLGEN) -I $(BUILD_SRC_DIR)
 Archive       = $(AR) $(AR.Flags)
-LArchive      = $(ToolDir)/llvm-ar rcsf
+LArchive      = $(LLVMToolDir)/llvm-ar rcsf
 ifdef RANLIB
 Ranlib        = $(RANLIB)
 else
@@ -466,7 +465,7 @@ LIBRARYNAME := $(strip $(LIBRARYNAME))
 LibName.LA := $(LibDir)/lib$(LIBRARYNAME).la
 LibName.A  := $(LibDir)/lib$(LIBRARYNAME).a
 LibName.O  := $(LibDir)/$(LIBRARYNAME).o
-LibName.BC := $(LibDir)/lib$(LIBRARYNAME).bc
+LibName.BCA:= $(LibDir)/lib$(LIBRARYNAME).bca
 
 #---------------------------------------------------------
 # Shared Library Targets:
@@ -511,37 +510,41 @@ endif
 #---------------------------------------------------------
 ifdef BYTECODE_LIBRARY
 
-ifdef EXPORTED_SYMBOL_LIST
-  BCLinkLib += -Xlinker -internalize-public-api-list=$(EXPORTED_SYMBOL_LIST)
-else
-  ifdef EXPORTED_SYMBOL_FILE
-    BCLinkLib += -Xlinker -internalize-public-api-file=$(EXPORTED_SYMBOL_FILE)
-  else
-    BCLinkLib += -Xlinker -disable-internalize
-  endif
-endif
+all-local:: $(LibName.BCA)
 
-all-local:: $(LibName.BC)
+ifdef EXPORTED_SYMBOL_FILE
+BCLinkLib = $(LLVMGCC) -shared -nostdlib -Xlinker \
+           -internalize-public-api-file=$(EXPORTED_SYMBOL_FILE)
 
-$(LibName.BC): $(BUILT_SOURCES) $(ObjectsBC) $(LibDir)/.dir $(GCCLD)
-       $(Echo) Linking $(BuildMode) Bytecode Library $(notdir $@)
-       $(Verb) $(BCLinkLib) -o $@ $(ObjectsBC)
+$(LibName.BCA): $(BUILT_SOURCES) $(ObjectsBC) $(LibDir)/.dir $(GCCLD)
+       $(Echo) Building $(BuildMode) Bytecode Archive $(notdir $@) \
+         "(internalize)"
+       $(BCLinkLib) -o $(ObjDir)/$(LIBRARYNAME).o $(ObjectsBC)
+       $(Verb) $(LArchive) $@ $(ObjDir)/$(LIBRARYNAME).o
+else
+$(LibName.BCA): $(BUILT_SOURCES) $(ObjectsBC) $(LibDir)/.dir $(GCCLD)
+       $(Echo) Building $(BuildMode) Bytecode Archive $(notdir $@)
+       $(Verb) $(LArchive) $@ $(ObjectsBC)
+
+endif
 
 clean-local::
-ifneq ($(strip $(LibName.BC)),)
-       -$(Verb) $(RM) -f $(LibName.BC)
+ifneq ($(strip $(LibName.BCA)),)
+       -$(Verb) $(RM) -f $(LibName.BCA)
 endif
 
-DestBytecodeLib = $(bytecode_libdir)/lib$(LIBRARYNAME).bc
+DestBytecodeLib = $(bytecode_libdir)/lib$(LIBRARYNAME).a
+
+install-bytecode: $(DestBytecodeLib)
 
 install-local:: $(DestBytecodeLib)
 
-$(DestBytecodeLib): $(bytecode_libdir) $(LibName.BC) 
-       $(Echo) Installing $(BuildMode) Bytecode Library $(DestBytecodeLib)
-       $(Verb) $(INSTALL) $(LibName.BC) $@
+$(DestBytecodeLib): $(bytecode_libdir) $(LibName.BCA
+       $(Echo) Installing $(BuildMode) Bytecode Archive $(DestBytecodeLib)
+       $(Verb) $(INSTALL) $(LibName.BCA) $@
 
 uninstall-local::
-       $(Echo) Uninstalling $(BuildMode) Bytecode Library $(DestBytecodeLib)
+       $(Echo) Uninstalling $(BuildMode) Bytecode Archive $(DestBytecodeLib)
        -$(Verb) $(RM) -f $(DestBytecodeLib)
 
 endif