[Beignet] [PATCH] Remove unncessary assertion in printf processing.

Yang, Rong R rong.r.yang at intel.com
Tue May 17 05:58:29 UTC 2016


Pushed.

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Pan, Xiuli
> Sent: Tuesday, May 3, 2016 14:43
> To: yan.wang at linux.intel.com; beignet at lists.freedesktop.org
> Subject: Re: [Beignet] [PATCH] Remove unncessary assertion in printf
> processing.
> 
> LGTM!
> Now conformance can pass.
> 
> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> yan.wang at linux.intel.com
> Sent: Tuesday, May 3, 2016 2:41 PM
> To: beignet at lists.freedesktop.org
> Cc: Yan Wang <yan.wang at linux.intel.com>
> Subject: [Beignet] [PATCH] Remove unncessary assertion in printf processing.
> 
> From: Yan Wang <yan.wang at linux.intel.com>
> 
> It causes alert when printf long vector.
> ---
>  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 51a1dab..7d21ebf 100644
> --- a/backend/src/llvm/llvm_gen_backend.cpp
> +++ b/backend/src/llvm/llvm_gen_backend.cpp
> @@ -4558,7 +4558,7 @@ namespace gbe
>                  uint32_t srcElemNum = 0;
>                  Value *srcValue = I.getOperand(n + 1);
>                  ir::Type srcType = getVectorInfo(ctx, srcValue, srcElemNum);
> -                GBE_ASSERT(!(srcType == ir::TYPE_S64 || srcType ==
> ir::TYPE_DOUBLE));
> +                GBE_ASSERT(!(srcType == ir::TYPE_DOUBLE));
> 
>                  uint32_t elemID = 0;
>                  for (elemID = 0; elemID < srcElemNum; ++elemID) {
> --
> 2.4.3
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/beignet
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list