[Mesa-dev] [PATCH] st/clover: Define __OPENCL_VERSION__ on the device side

[Vedran Miletić]

On 08/28/2016 04:42 PM, Niels Ole Salscheider wrote:
> This is required by the OpenCL standard.
> 
> Signed-off-by: Niels Ole Salscheider <niels_ole at salscheider-online.de>
Reviewed-by: Vedran Miletić <vedran at miletic.net>

Good catch. Do we miss more defines from [1]?

Regards,
Vedran

[1]
https://www.khronos.org/registry/cl/sdk/1.1/docs/man/xhtml/preprocessorDirectives.html

> ---
>  src/gallium/state_trackers/clover/llvm/invocation.cpp | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp b/src/gallium/state_trackers/clover/llvm/invocation.cpp
> index 5490d72..b5e8b52 100644
> --- a/src/gallium/state_trackers/clover/llvm/invocation.cpp
> +++ b/src/gallium/state_trackers/clover/llvm/invocation.cpp
> @@ -153,6 +153,9 @@ namespace {
>        // Add libclc include
>        c.getPreprocessorOpts().Includes.push_back("clc/clc.h");
>  
> +      // Add definition for the OpenCL version
> +      c.getPreprocessorOpts().addMacroDef("__OPENCL_VERSION__=110");
> +
>        // clc.h requires that this macro be defined:
>        c.getPreprocessorOpts().addMacroDef("cl_clang_storage_class_specifiers");
>        c.getPreprocessorOpts().addRemappedFile(
> 

-- 
Vedran Miletić
vedran.miletic.net