From: Philippe Waroquiers <philippe.waroquiers@skynet.be>
Date: Sat, 20 May 2017 17:00:39 +0000 (+0000)
Subject: Fix some warnings reported by PVS studio (see bug 379502)
X-Git-Tag: svn/VALGRIND_3_13_0^2~7
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9d87b60a402cd319819abe466804f303d3173861;p=thirdparty%2Fvalgrind.git

Fix some warnings reported by PVS studio (see bug 379502)


git-svn-id: svn://svn.valgrind.org/vex/trunk@3378
---

diff --git a/VEX/priv/host_arm64_isel.c b/VEX/priv/host_arm64_isel.c
index cc797eb464..50f9205d1d 100644
--- a/VEX/priv/host_arm64_isel.c
+++ b/VEX/priv/host_arm64_isel.c
@@ -732,7 +732,7 @@ Bool doHelperCall ( /*OUT*/UInt*   stackAdjustAfterCall,
    /* Do final checks, set the return values, and generate the call
       instruction proper. */
    vassert(nGSPTRs == 0 || nGSPTRs == 1);
-   vassert(nVECRETs == (retTy == Ity_V128 || retTy == Ity_V256) ? 1 : 0);
+   vassert(nVECRETs == ((retTy == Ity_V128 || retTy == Ity_V256) ? 1 : 0));
    vassert(*stackAdjustAfterCall == 0);
    vassert(is_RetLoc_INVALID(*retloc));
    switch (retTy) {
diff --git a/VEX/priv/host_arm_isel.c b/VEX/priv/host_arm_isel.c
index 2178ae45a3..3db44f4952 100644
--- a/VEX/priv/host_arm_isel.c
+++ b/VEX/priv/host_arm_isel.c
@@ -792,7 +792,7 @@ Bool doHelperCall ( /*OUT*/UInt*   stackAdjustAfterCall,
    /* Do final checks, set the return values, and generate the call
       instruction proper. */
    vassert(nGSPTRs == 0 || nGSPTRs == 1);
-   vassert(nVECRETs == (retTy == Ity_V128 || retTy == Ity_V256) ? 1 : 0);
+   vassert(nVECRETs == ((retTy == Ity_V128 || retTy == Ity_V256) ? 1 : 0));
    vassert(*stackAdjustAfterCall == 0);
    vassert(is_RetLoc_INVALID(*retloc));
    switch (retTy) {
diff --git a/VEX/priv/host_mips_isel.c b/VEX/priv/host_mips_isel.c
index 88f3708007..0dff8a67e1 100644
--- a/VEX/priv/host_mips_isel.c
+++ b/VEX/priv/host_mips_isel.c
@@ -605,7 +605,7 @@ static void doHelperCall(/*OUT*/UInt*   stackAdjustAfterCall,
    /* Do final checks, set the return values, and generate the call
       instruction proper. */
    vassert(nGSPTRs == 0 || nGSPTRs == 1);
-   vassert(nVECRETs == (retTy == Ity_V128 || retTy == Ity_V256) ? 1 : 0);
+   vassert(nVECRETs == ((retTy == Ity_V128 || retTy == Ity_V256) ? 1 : 0));
    vassert(*stackAdjustAfterCall == 0);
    vassert(is_RetLoc_INVALID(*retloc));
    switch (retTy) {