mgorny added inline comments.

================
Comment at: lib/Driver/ToolChains.cpp:4704
+      GetRuntimeLibType(Args) == ToolChain::RLT_CompilerRT)
+    CmdArgs.push_back("-lunwind");
+}
----------------
Hahnfeld wrote:
> As just written in D25008: This will probably result in problems if a system 
> default `libunwind.so.8` is installed...
I think this is the correct behavior in this case. Clang should work with 
whichever unwinder implementation is available, and if you want to change that, 
change the `libunwind.so` symlink.

However, I don't really think it's a good idea to have both unwinder libraries 
installed alongside, especially that they are sharing the same name.


https://reviews.llvm.org/D25402



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to