[Beignet] backend/gen_encoder.cpp: 2* bad asserts ?

Zhigang Gong zhigang.gong at linux.intel.com
Tue Apr 15 18:50:43 PDT 2014


Thanks for pointing this out, could you make a patch to fix it?
Thanks.

On Mon, Apr 14, 2014 at 09:46:38AM +0000, David Binderman wrote:
> hello there,
> 
> 1.
> 
> Release_v0.8/backend/src/backend/gen_encoder.cpp:446]: (warning) Logical disjunction always evaluates to true: elemNum>= 1 || elemNum <= 4.
> 
> Source code is
> 
>     assert(elemNum>= 1 || elemNum <= 4);
> 
> Suggest new code
> 
>     assert(elemNum>= 1 && elemNum <= 4);
> 
> 2.
> 
> Release_v0.8/backend/src/backend/gen_encoder.cpp:473]: (warning) Logical disjunction always evaluates to true: elemNum>= 1 || elemNum <= 4.
> 
> Duplicate.
> 
> Regards
> 
> David Binderman
> 
> 
> 
>  		 	   		  
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list