[Patch][OpenCL] CL2.0 atomic types

Anastasia Stulova anastasia.stulova at arm.com
Thu Feb 12 08:49:04 PST 2015


Hi Pekka,

Updated to include checks for extensions!

Does it look better now?

Thanks,
Anastasia

-----Original Message-----
From: Pekka Jääskeläinen [mailto:pekka.jaaskelainen at tut.fi] 
Sent: 14 January 2015 19:13
To: Anastasia Stulova; cfe-commits at cs.uiuc.edu
Subject: Re: [Patch][OpenCL] CL2.0 atomic types

Hi Anastasia,

 > I suggest patching first for the non-extension types and later add
support  > for the #pragma (and hopefully a check that allows customizing
the support  > per target).

How about this? I'm not sure if it'd nice to expose the atomics if they are
not supported by the target or enabled via the pragma.

On 01/13/2015 07:57 PM, Anastasia Stulova wrote:
> I just realized this was lost sometime during holidays. :)
>
> @Pekka, I don't have a fix for the extensions for the moment. Would it 
> still be ok if I commit this and we look into other bits later?
>
> -----Original Message-----
> From: Pekka Jääskeläinen [mailto:pekka.jaaskelainen at tut.fi]
> Sent: 15 December 2014 16:42
> To: Anastasia Stulova; cfe-commits at cs.uiuc.edu
> Subject: Re: [Patch][OpenCL] CL2.0 atomic types
>
> Hi,
>
> Looks good otherwise but (u)long and double atomics should be enabled 
> only if the cl_khr_int64_extended_atomics and 
> cl_khr_int64_base_atomics are enabled via #pragma, if I read the specs 
> correctly. The double atomics additionally require that the target
supports double.
>
> Similar applies to the pointer (difference) types (for 64b address 
> space, they are supported only if the above extensions are supported).
>
> Related to this is the question whether we should ask from the target 
> if the optional extensions should be supported or not, and not enable 
> them unconditionally.
>
> I suggest patching first for the non-extension types and later add 
> support for the #pragma (and hopefully a check that allows customizing 
> the support per target).
>
> On 12/15/2014 05:21 PM, Anastasia Stulova wrote:
>> Hello,
>>
>> This patch adds CL2.0 atomic types to Clang.
>>
>> Looking forward to your feedback,
>>
>> Anastasia
>
> --
> Pekka
>
>
>
>


--
--PJ
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cl20_atomic_types.patch
Type: application/octet-stream
Size: 9975 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150212/faa9ccd0/attachment.obj>


More information about the cfe-commits mailing list