<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 4, 2013 at 7:46 PM, Dag Lem <span dir="ltr"><<a href="mailto:dag@nimrod.no" target="_blank">dag@nimrod.no</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">Zhigang Gong <<a href="mailto:zhigang.gong@linux.intel.com">zhigang.gong@linux.intel.com</a>> writes:<br>

<br>
> And one minor comment below:<br>
<br>
</div>[...]<br>
<div class="im"><br>
>> +// FIXME:<br>
>> +// This is a transitional hack to bypass the LLVM 3.3 built-in types.<br>
>> +// See the Khronos SPIR specification for handling of these types.<br>
> Should not define the image address space again for 3.2/3.1.<br>
><br>
> #if 100 * __clang_major__ + __clang_minor__ == 303<br>
>> +#define __texture __attribute__((address_space(4)))<br>
> #endif<br>
<br>
</div>AFAIK it is legal to redefine a macro to the same definition. Does this<br>
cause any problems in the form of warnings or otherwise? As noted it is<br>
(hopefully) just a temporary hack for LLVM 3.3. I saw you mention that<br>
you're going to work on this - great! </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
In any case, feel free to change this to your liking. I haven't got<br>
access to my local git repository right now, so if you want me to change<br>
this it will have to wait a day or two.<br></blockquote><div><br></div><div style> You are right that it doesn't cause real problem. Just to make the code a little bit cleaner,</div><div style>  I will fix it. And no need to submit another version for this tiny fix.</div>
<div style><br></div><div style>  I'm still waiting for some comments/test report from other systems. Thanks for the patch.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888"><br>
--<br>
Best regards,<br>
<br>
Dag Lem<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Beignet mailing list<br>
<a href="mailto:Beignet@lists.freedesktop.org">Beignet@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/beignet" target="_blank">http://lists.freedesktop.org/mailman/listinfo/beignet</a><br>
</div></div></blockquote></div><br></div></div>