mgorny added a comment.

In https://reviews.llvm.org/D25008#555041, @rmaprath wrote:

> The patch generally makes sense to me.
>
> I wonder if the test configuration can also benefit from this; 
> `test/libcxx/test/target_info.py` manually updates the link line to use 
> `-lgcc_s` or `-lunwind` depending on how the build is configured. Presumably, 
> this workaround will no longer be needed if the linker script does that for 
> us?


I was originally thinking about that but we don't use the linker script on 
APPLE and I have no clue how that system is handled or whether it suffers from 
the same issue at all.

> (Note:- `-lgcc_s` is linked twice there to satisfy some weird dependency, 
> it's not a mistake)


Thanks for letting me know. I almost prepared a patch for that ;-D.


https://reviews.llvm.org/D25008



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

Reply via email to