<p dir="ltr">The only thing that I'm questioning here is that for radeonsi and newer nvidia chips, cards can have more than 4gb of memory, and I believe that the global address space for SI can/does include 64-bit pointers...  Sadly, I am not in a position where I can check at the moment.</p>
<p dir="ltr">--Aaron</p>
<div class="gmail_quote">On Nov 16, 2014 11:06 AM, "Jeroen Ketema" <<a href="mailto:j.ketema@imperial.ac.uk">j.ketema@imperial.ac.uk</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I’ll happily commit, but it would be good to get someone else’s feedback on this besides mine.<br>
<br>
Jeroen<br>
<br>
> On 02 Nov 2014, at 10:31, Jean-Sébastien Pédron <dumbbell@FreeBSD.org> wrote:<br>
><br>
> On 20.10.2014 12:03, Jeroen Ketema wrote:<br>
>>> On 18 Oct 2014, at 12:12, Jean-Sébastien Pédron <dumbbell@FreeBSD.org> wrote:<br>
>>><br>
>>> Including a standard or system header isn't allowed in OpenCL.<br>
>>><br>
>>> The type "size_t" needs to be explicitely defined now.<br>
>>><br>
>>> v2: Use __SIZE_TYPE__ instead of unsigned int.<br>
>>> v3: Define ptrdiff_t and NULL.<br>
>>> ---<br>
>>> generic/include/clc/clctypes.h | 7 +++++--<br>
>>> 1 file changed, 5 insertions(+), 2 deletions(-)<br>
>>><br>
>>> diff --git a/generic/include/clc/clctypes.h b/generic/include/clc/clctypes.h<br>
>>> index ca1372d..2e3db60 100644<br>
>>> --- a/generic/include/clc/clctypes.h<br>
>>> +++ b/generic/include/clc/clctypes.h<br>
>>> @@ -1,12 +1,13 @@<br>
>>> /* 6.1.1 Built-in Scalar Data Types */<br>
>>><br>
>>> -#include <stddef.h><br>
>>> -<br>
>>> typedef unsigned char uchar;<br>
>>> typedef unsigned short ushort;<br>
>>> typedef unsigned int uint;<br>
>>> typedef unsigned long ulong;<br>
>>><br>
>>> +typedef __SIZE_TYPE__ size_t;<br>
>>> +typedef __PTRDIFF_TYPE__ ptrdiff_t;<br>
>>> +<br>
>>> #define __stdint_join3(a,b,c) a ## b ## c<br>
>>><br>
>>> #define  __intn_t(n) __stdint_join3(__INT, n, _TYPE__)<br>
>>> @@ -84,3 +85,5 @@ typedef __attribute__((ext_vector_type(4))) double double4;<br>
>>> typedef __attribute__((ext_vector_type(8))) double double8;<br>
>>> typedef __attribute__((ext_vector_type(16))) double double16;<br>
>>> #endif<br>
>>> +<br>
>>> +#define NULL ((void *)0)<br>
>>> --<br>
>>> 2.1.0<br>
>><br>
>> LGTM.<br>
>><br>
>> Jeroen<br>
><br>
> If the patch is fine, can someone please commit it? :)<br>
><br>
> --<br>
> Jean-Sébastien Pédron<br>
><br>
> _______________________________________________<br>
> Libclc-dev mailing list<br>
> <a href="mailto:Libclc-dev@pcc.me.uk">Libclc-dev@pcc.me.uk</a><br>
> <a href="http://www.pcc.me.uk/cgi-bin/mailman/listinfo/libclc-dev" target="_blank">http://www.pcc.me.uk/cgi-bin/mailman/listinfo/libclc-dev</a><br>
<br>
<br>
_______________________________________________<br>
Libclc-dev mailing list<br>
<a href="mailto:Libclc-dev@pcc.me.uk">Libclc-dev@pcc.me.uk</a><br>
<a href="http://www.pcc.me.uk/cgi-bin/mailman/listinfo/libclc-dev" target="_blank">http://www.pcc.me.uk/cgi-bin/mailman/listinfo/libclc-dev</a><br>
</blockquote></div>