<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 15, 2015 at 9:18 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span><br>
>> - Marking of imported functions (for use in ThinLTO-specific symbol<br>
>> linking and global DCE, for example).<br>
><br>
> Marking how?  Do you mean giving them internal linkage, or something<br>
> else?<br>
<br>
</span>Mentioned just after this: either an in-memory flag on the Function<br>
class, or potentially in the IR. For the prototype I just had a flag<br>
on the Function class.</blockquote></span><div><br>Would this be anything other than "available externally" linkage? (either this module is responsible for emitting the definition of this function for other modules to call, or it's not - if it's not, either this module depends on another module to provide a definition (this would be "available externally") or it doesn't (this would be "internal" linkage")) - I think... though I'm sort of jumping into the middle of some of this & might've missed some context, sorry.</div></div></div></div></blockquote><div><br>& I see your follow up further down in that thread:<br><br>"<span style="font-family:monospace">There were only a couple minor tweaks required here (under the flag I</span><br style="font-family:monospace"><span style="font-family:monospace">added to the Function indicating that this was imported). Only</span><br style="font-family:monospace"><span style="font-family:monospace">promoted statics are remarked available_externally. For a</span><br style="font-family:monospace"><span style="font-family:monospace">non-discardable symbol that was imported, we can discard here since we</span><br style="font-family:monospace"><span style="font-family:monospace">are done with inlining (it is non-discardable in its home module)."<br></span><br>&, like Duncan, I'll wait for more details on that front. (may or may not be useful to split some of these subthreads into separate email threads to keep discussion clear - but I'm not sure)<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><span class=""><font color="#888888"><br><br>- David </font></span></div></div></div></div>
</blockquote></div><br></div></div>