On 7 November 2017 at 13:51, Pavel Labath <lab...@google.com> wrote: > On 7 November 2017 at 13:29, Zachary Turner <ztur...@google.com> wrote: >> If it works, is easy, and doesn’t regress anything I’d honestly rather just >> disable linker gc. > Ok, I'll try that. I'm starting not to to like this. Adding --whole-archive produces zillions of "multiple definitions of a symbol" errors on android. These can be squelched by telling the linker to ignore these errors (-z muldefs), but that's not really a supported configuration, and at that point this starts looking like a quite ugly hack.
I think we should just stick to the original proposal of ensuring the code is pulled in by actually introducing a dependency. If it turns out we need to do this in a lot of places, we can revisit this approach. _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
