[LLD] Cyclic dependency between ReaderWriter and YAML backend

[Shankar Easwaran]

Hi,

The actual problem was, lldYAML not being part of the linkline.

_*Commandline*_

/usr/bin/c++   -std=c++11 -fPIC -fvisibility-inlines-hidden -Wall -W 
-Wno-unused-parameter -Wwrite-strings -Wno-missing-field-initializers 
-pedantic -Wno-long-long -Wno-maybe-uninitialized -Wnon-virtual-dtor 
-fno-rtti CMakeFiles/linker-script-test.dir/linker-script-test.cpp.o  -o 
../../../../bin/linker-script-test  -lrt -ldl -lcurses -lpthread -lz 
../../../../lib/libLLVMSupport.so ../../../../lib/liblldReaderWriter.so 
../../../../lib/libLLVMObject.so ../../../../lib/liblldCore.so 
../../../../lib/libLLVMSupport.so ../../../../lib/liblldPasses.so -lrt 
-ldl -lcurses -lpthread -lz 
-Wl,-rpath,/usr2/seaswara/work/llvmtemp/llvm/llvm-build/lib

_*Fix*_

diff --git a/utils/linker-script-test/CMakeLists.txt 
b/utils/linker-script-test/CMakeLists.txt
index dd540a7..5b28a1a 100644
--- a/utils/linker-script-test/CMakeLists.txt
+++ b/utils/linker-script-test/CMakeLists.txt
@@ -5,4 +5,5 @@ add_lld_executable(linker-script-test
  target_link_libraries(linker-script-test
    LLVMSupport
    lldReaderWriter
+  lldYAML
    )

PS : If there is any sort of cyclic dependency, that should just be 
enclosed within --start-group/--end-group to avoid the dependency.

Thanks

Shankar Easwaran

On 9/2/2013 5:09 AM, Joerg Sonnenberger wrote:
> Hi all,
> when trying to build lld with BUILD_SHARED_LIBS=on, it fails due to a
> dependency cyclic between lib/ReaderWriter and lib/ReaderWriter/YAML.
> One possible solution is attached, but it is certainly not clean.
>
> Joerg
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits


-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130902/465437f0/attachment.html>