jprice added a subscriber: jprice.
jprice added a comment.

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

> In http://reviews.llvm.org/D18369#385799, @Anastasia wrote:
>
> > Is there any chance we could factor out the common bits into separate files 
> > to avoid large code repetition? I would imagine it should be quite doable 
> > as libs of each standard contain incremental changes.
>
>
> I saw some inconsistencies in the common part of the 1.2 and 2.0 headers. I 
> will try to consolidate them first then try to split.


Rather than having a separate header for each version with another header for 
the common parts, couldn't we just have a single header that uses #if guards 
for the version specific functionality? e.g.

  size_t __attribute__((overloadable)) get_global_id(uint dim);
  
  #if __OPENCL_C_VERSION__ >= 200
  size_t __attribute__((overloadable)) get_global_linear_id(void);
  #endif

This also allows an easy path for having OpenCL 1.1 and 1.0 support without 
having to add more copies of the header.


http://reviews.llvm.org/D18369



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to