<p dir="ltr">Rerunning the optimization pipeline after adding new function definitions will do bad things... It would cause the inliner and globalopt to make strange and suboptimal decisions....</p>
<div class="gmail_quote">On Feb 13, 2013 4:18 PM, "John McCall" <<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div>On Feb 13, 2013, at 3:40 PM, Nick Lewycky <<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>> wrote:</div><blockquote type="cite"><div dir="ltr">
On 13 February 2013 10:53, John McCall <span dir="ltr"><<a href="mailto:rjmccall@apple.com" target="_blank">rjmccall@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><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>On Feb 13, 2013, at 12:53 AM, Nick Lewycky <<a href="mailto:nlewycky@google.com" target="_blank">nlewycky@google.com</a>> wrote:<br>
> Thinking about this issue more, I'd like to retract this patch. Consider the case where we're implementing the rule that all ODR-used inline functions must have a definition. If Derived::fn is marked inline, we'll see a fictitious use of it and reject a valid translation unit.<br>



><br>
> My next plan is to remove this code from Sema entirely and do the equivalent work inside codegen proper. No reason it can't keep track of probable virtual function targets when writing out a virtual dispatch.<br>



<br>
</div>Ugh.  I am not thrilled by the idea of emitting massive amounts of code because something *might* devirtualize to it.<br>
<br>
Also, this potentially requires template instantiation.<br></blockquote><div><br></div><div>Me neither. However, Sema and CodeGen are already colluding to do this. See SemaExpr.cpp:11168 (MarkExprReferenced), which may trigger instantiation, and CGExprCXX.cpp:200 (EmitCXXMemberCallExpr) which assumes the potential callee was already instantiated.</div>


</div></div></div></blockquote><div><br></div>If we're willing to break some of the majestic beauty of the frontend-isolated optimizer, we could queue up a late module pass which</div><div>1) recognizes that we've emitted a reference to something that we're required to define in every translation unit that references it (easy to do with metadata),</div>
<div>2) goes back to Sema and IR-gen to produce that code, and</div><div>3) restarts the optimizer on the new functions.</div><div>Repeat until hopefully-rapid convergence.</div><div><br></div><div>John.</div></div>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div>