]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
rs6000: Change bitwise xor to an equality operator [PR106907]
authorJeevitha <jeevitha@linux.ibm.com>
Mon, 30 Oct 2023 09:07:07 +0000 (04:07 -0500)
committerJeevitha <jeevitha@linux.ibm.com>
Mon, 30 Oct 2023 10:38:19 +0000 (05:38 -0500)
PR106907 has a few warnings spotted from cppcheck. These warnings
are related to the need of precedence clarification. Instead of using xor,
it has been changed to equality check, which achieves the same result.
Additionally, comment indentation has been fixed.

2023-10-11  Jeevitha Palanisamy  <jeevitha@linux.ibm.com>

gcc/
PR target/106907
* config/rs6000/rs6000.cc (altivec_expand_vec_perm_const): Change bitwise
xor to an equality and fix comment indentation.

gcc/config/rs6000/rs6000.cc

index cc24dd5301e334f9e0136f747c8485b9ff7b4c9d..9f750674e53b516a8ecd6a06fd58761abdedea7e 100644 (file)
@@ -23647,10 +23647,10 @@ altivec_expand_vec_perm_const (rtx target, rtx op0, rtx op1,
                      && GET_MODE (XEXP (op0, 0)) != V8HImode)))
            continue;
 
-          /* For little-endian, the two input operands must be swapped
-             (or swapped back) to ensure proper right-to-left numbering
-             from 0 to 2N-1.  */
-         if (swapped ^ !BYTES_BIG_ENDIAN
+         /* For little-endian, the two input operands must be swapped
+            (or swapped back) to ensure proper right-to-left numbering
+            from 0 to 2N-1.  */
+         if (swapped == BYTES_BIG_ENDIAN
              && icode != CODE_FOR_vsx_xxpermdi_v16qi)
            std::swap (op0, op1);
          if (imode != V16QImode)