[PATCH] D25402: [Driver] Pass -lunwind when using libc++ + compiler-rt on Linux

[Michał Górny via cfe-commits]

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