Fair enough, originally I thought this was the “traditional” kind of GC, which would have made this approach simpler<br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 7, 2017 at 6:23 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 7 November 2017 at 13:51, Pavel Labath <<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>> wrote:<br>
> On 7 November 2017 at 13:29, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
>> If it works, is easy, and doesn’t regress anything I’d honestly rather just<br>
>> disable linker gc.<br>
> Ok, I'll try that.<br>
I'm starting not to to like this. Adding --whole-archive produces<br>
zillions of "multiple definitions of a symbol" errors on android.<br>
These can be squelched by telling the linker to ignore these errors<br>
(-z muldefs), but that's not really a supported configuration, and at<br>
that point this starts looking like a quite ugly hack.<br>
<br>
I think we should just stick to the original proposal of ensuring the<br>
code is pulled in by actually introducing a dependency. If it turns<br>
out we need to do this in a lot of places, we can revisit this<br>
approach.<br>
</blockquote></div>