<p dir="ltr">Hello,</p>
<p dir="ltr">That makes more sense to me sorry for the confusion. How should I go about implementing this?</p>
<p dir="ltr">Regards,<br>
Arthur</p>
<div class="gmail_quote">On Jul 4, 2014 11:52 PM, "Alp Toker" <<a href="mailto:alp@nuanti.com">alp@nuanti.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On 05/07/2014 00:44, Arthur Marble wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
That makes sense to me for other flags but in this case isn't clang not planning to support inline functions? If it is not then should we have the compiler spit out a message about how this is not used?<br>
</blockquote>
<br>
Hi Arthur,<br>
<br>
I believe the -finline-limit= warning is related to inliner decisions in the code generator backend, not the GNU nested function language extension.<br>
<br>
Alp.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Regards,<br>
Arthur<br>
<br>
<br>
On Fri, Jul 4, 2014 at 3:11 PM, Alp Toker <<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a> <mailto:<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>>> wrote:<br>
<br>
    Patch?<br>
<br>
    I investigated this flag too a while back and reached the<br>
    conclusion it's only interesting to support if we actually hook it<br>
    into the backend.<br>
<br>
    Otherwise it's unfair to users to silently claim support for an<br>
    option that has semantic significance, only to go and ignore it.<br>
    Especially if it's only to unblock a dozen packages it'd be a shame.<br>
<br>
    On the plus side, it was really rewarding to implement a<br>
    GCC-compatible -Wframe-larger-than= a few weeks ago so I *do*<br>
    think there's value in providing real implementations of these<br>
    flags because it informs decisions we need to make about<br>
    backend/frontend layering.<br>
<br>
    Alp.<br>
<br>
<br>
    On 04/07/2014 18:15, Rafael Ávila de Espíndola wrote:<br>
<br>
        adding cfe-commits<br>
<br>
        <a href="http://reviews.llvm.org/D4376" target="_blank">http://reviews.llvm.org/D4376</a><br>
<br>
<br>
<br>
        ______________________________<u></u>_________________<br>
        cfe-commits mailing list<br>
        <a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a> <mailto:<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.<u></u>edu</a>><br>
        <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-commits</a><br>
<br>
<br>
    --     <a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
    the browser experts<br>
<br>
<br>
</blockquote>
<br>
-- <br>
<a href="http://www.nuanti.com" target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</blockquote></div>