<font size="2"><font face="tahoma,sans-serif">Hi;<br></font></font><br><div class="gmail_quote">On Thu, Dec 1, 2011 at 3:46 PM, Brian Paul <span dir="ltr">&lt;<a href="mailto:brianp@vmware.com">brianp@vmware.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">On 12/01/2011 07:36 AM, İsmail Dönmez wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Hi;<br>
<br>
On Wed, Nov 9, 2011 at 8:08 PM, İsmail Dönmez &lt;<a href="mailto:ismail@namtrac.org" target="_blank">ismail@namtrac.org</a><br></div><div class="im">
&lt;mailto:<a href="mailto:ismail@namtrac.org" target="_blank">ismail@namtrac.org</a>&gt;&gt; wrote:<br>
<br>
    Another thing to notice that nested functions require executable<br>
    stack. This is also another reason to get rid of them.<br>
<br>
    On Wed, Nov 9, 2011 at 3:11 PM, İsmail Dönmez &lt;<a href="mailto:ismail@namtrac.org" target="_blank">ismail@namtrac.org</a><br></div><div class="im">
    &lt;mailto:<a href="mailto:ismail@namtrac.org" target="_blank">ismail@namtrac.org</a>&gt;&gt; wrote:<br>
<br>
        Hi;<br>
<br>
        nouveau_array.c seems to be using nested functions which is<br>
        not supported by clang (or the EDG based compilers it seems),<br>
        the reduced testcase looks like this:<br>
<br>
        foo()<br>
        {<br>
             auto void f();<br>
             void f() {};<br>
        }<br>
<br>
        I talked to the clang developers and they don&#39;t plan to<br>
        support this GNU extension since (they say) it would be too<br>
        hard to support it in the parser. Clang website notes the<br>
        following about this issue (from<br>
        <a href="http://clang.llvm.org/docs/UsersManual.html#c_unimpl_gcc" target="_blank">http://clang.llvm.org/docs/<u></u>UsersManual.html#c_unimpl_gcc</a> )<br>
<br>
        &lt;quote&gt;<br>
        clang does not support nested functions; this is a complex<br>
        feature which is infrequently used, so it is unlikely to be<br>
        implemented anytime soon.<br>
        &lt;/quote&gt;<br>
<br>
        Is there any chance of removing such code from Mesa?<br>
<br>
<br>
Would be nice to get a comment on this.<br>
</div></blockquote>
<br>
I imagine you&#39;re the first person to try to compile that driver with clang because nobody else has mentioned this before.<br>
<br>
I don&#39;t think anyone would object to the code being rewritten to avoid nested functions.  Is that something that you could do?</blockquote><div><br></div><div>I imagined I had to the real work ;) I&#39;ll see what I can do.</div>

<div><br></div><div>Regards,</div><div>ismail</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><span class="HOEnZb"> </span></blockquote></div>