diff options
| author | Jeevitha <jeevitha@linux.ibm.com> | 2023-10-30 04:07:07 -0500 |
|---|---|---|
| committer | Jeevitha <jeevitha@linux.ibm.com> | 2023-10-30 05:38:19 -0500 |
| commit | 36a52cdc23383e51359630e566b62fa62011428d (patch) | |
| tree | 9c53392d148565a5518b9306a1c488602820da62 /gcc | |
| parent | ff4cea05a6e6d034eed5212109133c4b5da8520c (diff) | |
| download | gcc-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.
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/config/rs6000/rs6000.cc | 8 |
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) |