Re: coverage, propagate visibility

2011-11-15 Thread Nathan Sidwell
On 11/15/11 07:55, Markus Trippelsdorf wrote: Nothing changed the set of emitted functions. The error message above was just an example. Will post the full error message in the PR (51113). I will also try to come up with a smaller testcase. Something's changed the linkage of ZN2js11WeakMapBase

Re: coverage, propagate visibility

2011-11-14 Thread Markus Trippelsdorf
On 2011.11.15 at 07:09 +, Nathan Sidwell wrote: > On 11/14/11 22:33, Markus Trippelsdorf wrote: > > On 2011.11.14 at 21:59 +, Nathan Sidwell wrote: > >> This patch propagates visibility information in the same way that weakness > >> is > >> for externally visible coverage objects. I don't

Re: coverage, propagate visibility

2011-11-14 Thread Nathan Sidwell
On 11/14/11 22:33, Markus Trippelsdorf wrote: On 2011.11.14 at 21:59 +, Nathan Sidwell wrote: This patch propagates visibility information in the same way that weakness is for externally visible coverage objects. I don't know whether this will fix the reported firefox build failure -- can y

Re: coverage, propagate visibility

2011-11-14 Thread Markus Trippelsdorf
On 2011.11.14 at 21:59 +, Nathan Sidwell wrote: > This patch propagates visibility information in the same way that weakness is > for externally visible coverage objects. I don't know whether this will fix > the > reported firefox build failure -- can you try? The patch doesn't help unfor

coverage, propagate visibility

2011-11-14 Thread Nathan Sidwell
This patch propagates visibility information in the same way that weakness is for externally visible coverage objects. I don't know whether this will fix the reported firefox build failure -- can you try? Even if it doesn't, I'm pretty sure it's an improvement, but I'll leave it for comment be