<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Aug 28, 2015, at 10:26 AM, Jake VanAdrighem <<a href="mailto:jvanadrighem@gmail.com" class="">jvanadrighem@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><br class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Fri, Aug 28, 2015 at 9:43 AM, Adrian Prantl via llvm-commits <span dir="ltr" class=""><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" class="">llvm-commits@lists.llvm.org</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aprantl added a comment.<br class="">
<br class="">
I assume that a function without code will never get inlined, so this should not cause a problem with inlining.<br class="">
David, could this affect overload resolution if it is a C++ method that is being eliminated?<br class="">
<span class=""><br class=""></span></blockquote><div class=""><br class=""></div><div class="">I thought we did inline empty functions? I know that in analyzeCall we give the go ahead if the function is empty.</div></div></div></div></div></blockquote><div><br class=""></div><div>Yes, what I should have said is that no instruction (and thus no insn without a DebugLoc) from the inlined function will make it into the caller if the inlined function is empty.</div><div>-- adrian</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
<br class="">
Repository:<br class="">
rL LLVM<br class="">
<br class="">
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D12426&d=BQMFaQ&c=eEvniauFctOgLOKGJOplqw&r=cTx6f1tAfqPeajYunFWp7_8ot79RnHyNteqzig4fXmA&m=RALQ1lbSTJQTonC5PrWB7i06aNPRZShsFzrhuaEf6uI&s=tI-l0dE_8aTjuPkpmCYdAQB5WEW6dA-gcY7s_dPg_2E&e=" rel="noreferrer" target="_blank" class="">http://reviews.llvm.org/D12426</a><br class="">
<br class="">
<br class="">
<br class="">
</span>_______________________________________________<br class="">
llvm-commits mailing list<br class="">
<a href="mailto:llvm-commits@lists.llvm.org" class="">llvm-commits@lists.llvm.org</a><br class="">
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_llvm-2Dcommits&d=BQMFaQ&c=eEvniauFctOgLOKGJOplqw&r=cTx6f1tAfqPeajYunFWp7_8ot79RnHyNteqzig4fXmA&m=RALQ1lbSTJQTonC5PrWB7i06aNPRZShsFzrhuaEf6uI&s=plUUL48mbaDlt0y0IzmYE4KGn94oqRooYnSMhuT4qo0&e=" rel="noreferrer" target="_blank" class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="">
</blockquote></div><br class=""></div></div>
</div></blockquote></div><br class=""></body></html>