<font size="2"><font face="tahoma,sans-serif">Hi;<br></font></font><br><div class="gmail_quote">On Wed, Nov 9, 2011 at 8:08 PM, İsmail Dönmez <span dir="ltr">&lt;<a href="mailto:ismail@namtrac.org">ismail@namtrac.org</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><font size="2"><font face="tahoma,sans-serif">Another thing to notice that nested functions require executable stack. This is also another reason to get rid of them.<br>

</font></font><div class="HOEnZb"><div class="h5"><br><div class="gmail_quote">On Wed, Nov 9, 2011 at 3:11 PM, İsmail Dönmez <span dir="ltr">&lt;<a href="mailto:ismail@namtrac.org" target="_blank">ismail@namtrac.org</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><font size="2"><font face="tahoma,sans-serif">Hi;<br><br>nouveau_array.c seems to be using nested functions which is not supported by clang (or the EDG based compilers it seems), 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 support this GNU extension since (they say) it would be too hard to support it in the parser. Clang website notes the following about this issue (from <a href="http://clang.llvm.org/docs/UsersManual.html#c_unimpl_gcc" target="_blank">http://clang.llvm.org/docs/UsersManual.html#c_unimpl_gcc</a> )<br>



</font></font><br>&lt;quote&gt;<br>clang does not support nested functions; this is a complex feature which is infrequently used, so it is unlikely to be implemented anytime soon.<br>&lt;/quote&gt;<br><br>Is there any chance of removing such code from Mesa?<br>

</blockquote></div></div></div></blockquote><div><br></div><div>Would be nice to get a comment on this. </div><div><br></div><div>Thanks,</div><div>ismail</div><div><br></div></div><br>