|
From: | Richard Henderson |
Subject: | Re: [Qemu-devel] [PULL 08/16] tcg/i386: Support vector comparison select value |
Date: | Thu, 30 May 2019 07:50:03 -0500 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 5/30/19 6:26 AM, Peter Maydell wrote: >> -#define TCG_TARGET_HAS_cmpsel_vec 0 >> +#define TCG_TARGET_HAS_cmpsel_vec -1 > > This is the only place where we define a TCG_TARGET_HAS_* macro > to something other than 0 or 1, which means that Coverity > complains (CID 1401702) when we use it in a logical boolean expression > return have_vec && TCG_TARGET_HAS_cmpsel_vec; > later on. > > Should it really be -1, or is this a typo for 1 ? It really should be -1. See commit 25c012b4009256505be3430480954a0233de343e, which contains the rationale. r~
[Prev in Thread] | Current Thread | [Next in Thread] |