aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeevitha <jeevitha@linux.ibm.com>2023-10-30 04:07:07 -0500
committerJeevitha <jeevitha@linux.ibm.com>2023-10-30 05:38:19 -0500
commit36a52cdc23383e51359630e566b62fa62011428d (patch)
tree9c53392d148565a5518b9306a1c488602820da62
parentff4cea05a6e6d034eed5212109133c4b5da8520c (diff)
downloadgcc-36a52cdc23383e51359630e566b62fa62011428d.zip
gcc-36a52cdc23383e51359630e566b62fa62011428d.tar.gz
gcc-36a52cdc23383e51359630e566b62fa62011428d.tar.bz2
rs6000: Change bitwise xor to an equality operator [PR106907]
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.
-rw-r--r--gcc/config/rs6000/rs6000.cc8
1 files changed, 4 insertions, 4 deletions
diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc
index cc24dd5..9f75067 100644
--- a/gcc/config/rs6000/rs6000.cc
+++ b/gcc/config/rs6000/rs6000.cc
@@ -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)