<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">&gt; Attached patch changes __retval variable to _ret_val. According to GLSL<br>

&gt; spec, &quot;all identifiers containing two consecutive underscores are reserved<br>
&gt; as possible future keywords&quot;. Windows Radeon 5xxx drivers actually complain<br>
&gt; about this case.<br>
<br>
</div>You definitely need something to process names to produce unique, safe<br>
names for usage in your GLSL-to-GLSL compiler.  The names used<br>
internally in the compiler are not unique or safe to use as identifiers<br>
in GLSL.<br></blockquote><div><br></div><div>Yeah, I realized that patch is not directly related to Mesa just after sending it. Oh well.</div><div><br></div><div>Otherwise, I do explicitly unique-ify the names in my GLSL-to-GLSL compiler fork (e.g. append unique postfixes to local variables after inlining, etc.). I haven&#39;t ran into other issues yet, except this double underscore thing in __retval.</div>
</div><br clear="all"><br>-- <br>Aras Pranckevičius<br>work: <a href="http://unity3d.com">http://unity3d.com</a><br>home: <a href="http://aras-p.info">http://aras-p.info</a><br>
<div><br></div>