[Beignet] [PATCH 1/2] GBE: Fix kernel argument size for vector3

[Zhigang Gong]

This patchset LGTM, will push latter, thanks.

On Thu, Nov 06, 2014 at 03:44:48PM +0800, Ruiling Song wrote:
> per OpenCL Spec 1.2, see 6.1.5 Alignment of Types.
> For 3-component vector data types, the size of the data type
> is 4 * sizeof(component).
> 
> Signed-off-by: Ruiling Song <ruiling.song at intel.com>
> ---
>  backend/src/llvm/llvm_gen_backend.cpp |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/backend/src/llvm/llvm_gen_backend.cpp b/backend/src/llvm/llvm_gen_backend.cpp
> index bb2c1dd..b1b9382 100644
> --- a/backend/src/llvm/llvm_gen_backend.cpp
> +++ b/backend/src/llvm/llvm_gen_backend.cpp
> @@ -1399,7 +1399,7 @@ namespace gbe
>            const uint32_t elemSize = getTypeByteSize(unit, elemType);
>            const uint32_t elemNum = vectorType->getNumElements();
>            //vector's elemType always scalar type
> -          ctx.input(argName, ir::FunctionArgument::VALUE, reg, llvmInfo, elemNum*elemSize, getAlignmentByte(unit, type), 0);
> +          ctx.input(argName, ir::FunctionArgument::VALUE, reg, llvmInfo, getTypeByteSize(unit, type), getAlignmentByte(unit, type), 0);
>  
>            ir::Function& fn = ctx.getFunction();
>            for(uint32_t i=1; i < elemNum; i++) {
> -- 
> 1.7.10.4
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet