[PATCH] D18369: [OpenCL] Upstreaming khronos OpenCL 1.2/2.0 header files.

Anastasia Stulova via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 6 09:16:55 PDT 2016


Anastasia added a comment.

In http://reviews.llvm.org/D18369#393323, @yaxunl wrote:

> We need to discuss the layout of the header file.
>
> For the builtin functions, I plan to follow the order of the spec and extension spec. If there are difference in 1.2 and 2.0, use condition macro.
>
> For functions with double or half arguments, we have two options:
>
> 1. keep all functions with double arguments as one section, keep all functions with half arguments as another section. This is the current layout of opencl-20.h
> 2. keep functions with the same name together so that they follow the spec order.
>
>   Any suggestions? Thanks.


I am fine with either option actually.

Thanks,
Anastasia


http://reviews.llvm.org/D18369





More information about the cfe-commits mailing list