[Mesa-dev] [PATCH] swr/rast: Fix llvm5 behavior

Cherniak, Bruce bruce.cherniak at intel.com
Fri Jan 19 23:06:25 UTC 2018


Reviewed-by: Bruce Cherniak <bruce.cherniak at intel.com> 

> On Jan 19, 2018, at 5:03 PM, George Kyriazis <george.kyriazis at intel.com> wrote:
> 
> For some reason llvm5 is picky about accepting a void * type in the
> case of building an argument list.
> 
> Since we don't care about the type (we ignore the argument for now),
> pick another pointer type
> ---
> src/gallium/drivers/swr/rasterizer/jitter/JitManager.cpp | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/gallium/drivers/swr/rasterizer/jitter/JitManager.cpp b/src/gallium/drivers/swr/rasterizer/jitter/JitManager.cpp
> index a9a416a..93e3240 100644
> --- a/src/gallium/drivers/swr/rasterizer/jitter/JitManager.cpp
> +++ b/src/gallium/drivers/swr/rasterizer/jitter/JitManager.cpp
> @@ -128,7 +128,8 @@ JitManager::JitManager(uint32_t simdWidth, const char *arch, const char* core)
> #endif
>     std::vector<Type*> fsArgs;
> 
> -    fsArgs.push_back(PointerType::get(Type::getVoidTy(mContext), 0));
> +    // llvm5 is picky and does not take a void * type
> +    fsArgs.push_back(PointerType::get(Gen_SWR_FETCH_CONTEXT(this), 0));
> 
>     fsArgs.push_back(PointerType::get(Gen_SWR_FETCH_CONTEXT(this), 0));
> #if USE_SIMD16_SHADERS
> -- 
> 2.7.4
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev



More information about the mesa-dev mailing list