r273031 - NVPTX: Add supported CL features

2016-06-17 Thread Jan Vesely via cfe-commits
Author: jvesely Date: Fri Jun 17 14:02:14 2016 New Revision: 273031 URL: http://llvm.org/viewvc/llvm-project?rev=273031&view=rev Log: NVPTX: Add supported CL features Fixes libclc compilation broken by r269670 Reviewers: jholewinsky Differential Revision: http://reviews.llvm.org/D20389 Added:

Re: [PATCH] D20389: NVPTX: Add supported CL features

2016-06-17 Thread Jan Vesely via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL273031: NVPTX: Add supported CL features (authored by jvesely). Changed prior to commit: http://reviews.llvm.org/D20389?vs=60257&id=61119#toc Repository: rL LLVM http://reviews.llvm.org/D20389 File

[libclc] r273044 - configure: Remove device specific defines

2016-06-17 Thread Jan Vesely via cfe-commits
Author: jvesely Date: Fri Jun 17 15:30:50 2016 New Revision: 273044 URL: http://llvm.org/viewvc/llvm-project?rev=273044&view=rev Log: configure: Remove device specific defines Signed-off-by: Jan Vesely Reviewed-by: Tom Stellard Modified: libclc/trunk/configure.py Modified: libclc/trunk/co

[libclc] r273042 - 64 bit integers are legal in full profile without an extension

2016-06-17 Thread Jan Vesely via cfe-commits
Author: jvesely Date: Fri Jun 17 15:30:41 2016 New Revision: 273042 URL: http://llvm.org/viewvc/llvm-project?rev=273042&view=rev Log: 64 bit integers are legal in full profile without an extension Signed-off-by: Jan Vesely Reviewed-by: Tom Stellard Modified: libclc/trunk/configure.py l

[libclc] r273043 - nvptx: Drop feature defines.

2016-06-17 Thread Jan Vesely via cfe-commits
Author: jvesely Date: Fri Jun 17 15:30:49 2016 New Revision: 273043 URL: http://llvm.org/viewvc/llvm-project?rev=273043&view=rev Log: nvptx: Drop feature defines. This is now handled by clang Signed-off-by: Jan Vesely Reviewed-by: Tom Stellard Modified: libclc/trunk/configure.py Modified

Re: [PATCH] D19932: [OpenCL] Add to_{global|local|private} builtin functions.

2016-06-22 Thread Jan Vesely via cfe-commits
jvesely added a subscriber: jvesely. jvesely added a comment. Why couldn't this be declared in CLC header? There are multiple instances of declaring functions for all available types in libclc (see for example convert functions in convert.h). Repository: rL LLVM http://reviews.llvm.org/D199

Re: [PATCH] D19932: [OpenCL] Add to_{global|local|private} builtin functions.

2016-06-23 Thread Jan Vesely via cfe-commits
jvesely added a comment. In http://reviews.llvm.org/D19932#465763, @cfe-commits wrote: > The returned pointer should point to the same pointee type as the argument. > Header file cannot guarantee that. > > Sam how come? is there a possibility to have two different types using the same name?

Re: [PATCH] D19932: [OpenCL] Add to_{global|local|private} builtin functions.

2016-06-23 Thread Jan Vesely via cfe-commits
jvesely added a comment. In http://reviews.llvm.org/D19932#465784, @yaxunl wrote: > In http://reviews.llvm.org/D19932#465781, @jvesely wrote: > > > In http://reviews.llvm.org/D19932#465763, @cfe-commits wrote: > > > > > The returned pointer should point to the same pointee type as the > > > argu

Re: [PATCH] D19932: [OpenCL] Add to_{global|local|private} builtin functions.

2016-06-23 Thread Jan Vesely via cfe-commits
jvesely added a comment. In http://reviews.llvm.org/D19932#465807, @yaxunl wrote: > > this is not true. as I pointed out earlier, take a look at libclc headers. > > a lot functions are defined for multiple types while maintaining type > > safety. > > > there is no problem having TYPE * to_glob

Re: [PATCH] D21031: [OpenCL] Allow -cl-std and other standard -cl- options in driver

2016-07-01 Thread Jan Vesely via cfe-commits
jvesely added a subscriber: jvesely. Comment at: cfe/trunk/include/clang/Driver/Options.td:381 @@ +380,3 @@ +def cl_unsafe_math_optimizations : Flag<["-"], "cl-unsafe-math-optimizations">, Group, Flags<[CC1Option]>, + HelpText<"OpenCL only. Allow unsafe floating-point optimizati

r275030 - AMDGPU: Export workitem builtins

2016-07-10 Thread Jan Vesely via cfe-commits
Author: jvesely Date: Sun Jul 10 17:38:04 2016 New Revision: 275030 URL: http://llvm.org/viewvc/llvm-project?rev=275030&view=rev Log: AMDGPU: Export workitem builtins Reviewers: tstellardAMD Differential Revision: http://reviews.llvm.org/D20299 Modified: cfe/trunk/include/clang/Basic/Builti

<    1   2   3   4