[PATCH] fpu/softfloat.c: add spaces around operator

Xinhao Zhang posted 1 patch 3 years, 5 months ago
Test checkpatch failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201114130715.1126922-1-zhangxinhao1@huawei.com
Maintainers: "Alex Bennée" <alex.bennee@linaro.org>, Peter Maydell <peter.maydell@linaro.org>, Aurelien Jarno <aurelien@aurel32.net>
fpu/softfloat.c | 150 ++++++++++++++++++++++++------------------------
1 file changed, 75 insertions(+), 75 deletions(-)
[PATCH] fpu/softfloat.c: add spaces around operator
Posted by Xinhao Zhang 3 years, 5 months ago
Fix code style. Operator needs spaces both sides.

Signed-off-by: Xinhao Zhang <zhangxinhao1@huawei.com>
Signed-off-by: Kai Deng <dengkai1@huawei.com>
---
 fpu/softfloat.c | 150 ++++++++++++++++++++++++------------------------
 1 file changed, 75 insertions(+), 75 deletions(-)

diff --git a/fpu/softfloat.c b/fpu/softfloat.c
index 67cfa0fd82..9938a20905 100644
--- a/fpu/softfloat.c
+++ b/fpu/softfloat.c
@@ -3786,13 +3786,13 @@ static int32_t roundAndPackInt32(bool zSign, uint64_t absZ,
         abort();
     }
     roundBits = absZ & 0x7F;
-    absZ = ( absZ + roundIncrement )>>7;
+    absZ = ( absZ + roundIncrement ) >> 7;
     if (!(roundBits ^ 0x40) && roundNearestEven) {
         absZ &= ~1;
     }
     z = absZ;
     if ( zSign ) z = - z;
-    if ( ( absZ>>32 ) || ( z && ( ( z < 0 ) ^ zSign ) ) ) {
+    if ( ( absZ >> 32 ) || ( z && ( ( z < 0 ) ^ zSign ) ) ) {
         float_raise(float_flag_invalid, status);
         return zSign ? INT32_MIN : INT32_MAX;
     }
@@ -3938,7 +3938,7 @@ static void
     int8_t shiftCount;
 
     shiftCount = clz32(aSig) - 8;
-    *zSigPtr = aSig<<shiftCount;
+    *zSigPtr = aSig << shiftCount;
     *zExpPtr = 1 - shiftCount;
 
 }
@@ -4033,7 +4033,7 @@ static float32 roundAndPackFloat32(bool zSign, int zExp, uint32_t zSig,
     if (roundBits) {
         status->float_exception_flags |= float_flag_inexact;
     }
-    zSig = ( zSig + roundIncrement )>>7;
+    zSig = ( zSig + roundIncrement ) >> 7;
     if (!(roundBits ^ 0x40) && roundNearestEven) {
         zSig &= ~1;
     }
@@ -4058,7 +4058,7 @@ static float32
     int8_t shiftCount;
 
     shiftCount = clz32(zSig) - 1;
-    return roundAndPackFloat32(zSign, zExp - shiftCount, zSig<<shiftCount,
+    return roundAndPackFloat32(zSign, zExp - shiftCount, zSig << shiftCount,
                                status);
 
 }
@@ -4076,7 +4076,7 @@ static void
     int8_t shiftCount;
 
     shiftCount = clz64(aSig) - 11;
-    *zSigPtr = aSig<<shiftCount;
+    *zSigPtr = aSig << shiftCount;
     *zExpPtr = 1 - shiftCount;
 
 }
@@ -4096,7 +4096,7 @@ static inline float64 packFloat64(bool zSign, int zExp, uint64_t zSig)
 {
 
     return make_float64(
-        ( ( (uint64_t) zSign )<<63 ) + ( ( (uint64_t) zExp )<<52 ) + zSig);
+        ( ( (uint64_t) zSign ) << 63 ) + ( ( (uint64_t) zExp ) << 52 ) + zSig);
 
 }
 
@@ -4189,7 +4189,7 @@ static float64 roundAndPackFloat64(bool zSign, int zExp, uint64_t zSig,
     if (roundBits) {
         status->float_exception_flags |= float_flag_inexact;
     }
-    zSig = ( zSig + roundIncrement )>>10;
+    zSig = ( zSig + roundIncrement ) >> 10;
     if (!(roundBits ^ 0x200) && roundNearestEven) {
         zSig &= ~1;
     }
@@ -4214,7 +4214,7 @@ static float64
     int8_t shiftCount;
 
     shiftCount = clz64(zSig) - 1;
-    return roundAndPackFloat64(zSign, zExp - shiftCount, zSig<<shiftCount,
+    return roundAndPackFloat64(zSign, zExp - shiftCount, zSig << shiftCount,
                                status);
 
 }
@@ -4232,7 +4232,7 @@ void normalizeFloatx80Subnormal(uint64_t aSig, int32_t *zExpPtr,
     int8_t shiftCount;
 
     shiftCount = clz64(aSig);
-    *zSigPtr = aSig<<shiftCount;
+    *zSigPtr = aSig << shiftCount;
     *zExpPtr = 1 - shiftCount;
 }
 
@@ -4326,7 +4326,7 @@ floatx80 roundAndPackFloatx80(int8_t roundingPrecision, bool zSign,
             zSig0 += roundIncrement;
             if ( (int64_t) zSig0 < 0 ) zExp = 1;
             roundIncrement = roundMask + 1;
-            if ( roundNearestEven && ( roundBits<<1 == roundIncrement ) ) {
+            if ( roundNearestEven && ( roundBits << 1 == roundIncrement ) ) {
                 roundMask |= roundIncrement;
             }
             zSig0 &= ~ roundMask;
@@ -4342,7 +4342,7 @@ floatx80 roundAndPackFloatx80(int8_t roundingPrecision, bool zSign,
         zSig0 = UINT64_C(0x8000000000000000);
     }
     roundIncrement = roundMask + 1;
-    if ( roundNearestEven && ( roundBits<<1 == roundIncrement ) ) {
+    if ( roundNearestEven && ( roundBits << 1 == roundIncrement ) ) {
         roundMask |= roundIncrement;
     }
     zSig0 &= ~ roundMask;
@@ -4509,7 +4509,7 @@ static inline uint64_t extractFloat128Frac0( float128 a )
 static inline int32_t extractFloat128Exp( float128 a )
 {
 
-    return ( a.high>>48 ) & 0x7FFF;
+    return ( a.high >> 48 ) & 0x7FFF;
 
 }
 
@@ -4546,11 +4546,11 @@ static void
     if ( aSig0 == 0 ) {
         shiftCount = clz64(aSig1) - 15;
         if ( shiftCount < 0 ) {
-            *zSig0Ptr = aSig1>>( - shiftCount );
-            *zSig1Ptr = aSig1<<( shiftCount & 63 );
+            *zSig0Ptr = aSig1 >> ( - shiftCount );
+            *zSig1Ptr = aSig1 << ( shiftCount & 63 );
         }
         else {
-            *zSig0Ptr = aSig1<<shiftCount;
+            *zSig0Ptr = aSig1 << shiftCount;
             *zSig1Ptr = 0;
         }
         *zExpPtr = - shiftCount - 63;
@@ -4775,7 +4775,7 @@ floatx80 int32_to_floatx80(int32_t a, float_status *status)
     absA = zSign ? - a : a;
     shiftCount = clz32(absA) + 32;
     zSig = absA;
-    return packFloatx80( zSign, 0x403E - shiftCount, zSig<<shiftCount );
+    return packFloatx80( zSign, 0x403E - shiftCount, zSig << shiftCount );
 
 }
 
@@ -4797,7 +4797,7 @@ float128 int32_to_float128(int32_t a, float_status *status)
     absA = zSign ? - a : a;
     shiftCount = clz32(absA) + 17;
     zSig0 = absA;
-    return packFloat128( zSign, 0x402E - shiftCount, zSig0<<shiftCount, 0 );
+    return packFloat128( zSign, 0x402E - shiftCount, zSig0 << shiftCount, 0 );
 
 }
 
@@ -4818,7 +4818,7 @@ floatx80 int64_to_floatx80(int64_t a, float_status *status)
     zSign = ( a < 0 );
     absA = zSign ? - a : a;
     shiftCount = clz64(absA);
-    return packFloatx80( zSign, 0x403E - shiftCount, absA<<shiftCount );
+    return packFloatx80( zSign, 0x403E - shiftCount, absA << shiftCount );
 
 }
 
@@ -4901,7 +4901,7 @@ floatx80 float32_to_floatx80(float32 a, float_status *status)
         normalizeFloat32Subnormal( aSig, &aExp, &aSig );
     }
     aSig |= 0x00800000;
-    return packFloatx80( aSign, aExp + 0x3F80, ( (uint64_t) aSig )<<40 );
+    return packFloatx80( aSign, aExp + 0x3F80, ( (uint64_t) aSig ) << 40 );
 
 }
 
@@ -4933,7 +4933,7 @@ float128 float32_to_float128(float32 a, float_status *status)
         normalizeFloat32Subnormal( aSig, &aExp, &aSig );
         --aExp;
     }
-    return packFloat128( aSign, aExp + 0x3F80, ( (uint64_t) aSig )<<25, 0 );
+    return packFloat128( aSign, aExp + 0x3F80, ( (uint64_t) aSig ) << 25, 0 );
 
 }
 
@@ -4997,10 +4997,10 @@ float32 float32_rem(float32 a, float32 b, float_status *status)
         q = ( bSig <= aSig );
         if ( q ) aSig -= bSig;
         if ( 0 < expDiff ) {
-            q = ( ( (uint64_t) aSig )<<32 ) / bSig;
+            q = ( ( (uint64_t) aSig ) << 32 ) / bSig;
             q >>= 32 - expDiff;
             bSig >>= 2;
-            aSig = ( ( aSig>>1 )<<( expDiff - 1 ) ) - bSig * q;
+            aSig = ( ( aSig >> 1 ) << ( expDiff - 1 ) ) - bSig * q;
         }
         else {
             aSig >>= 2;
@@ -5009,21 +5009,21 @@ float32 float32_rem(float32 a, float32 b, float_status *status)
     }
     else {
         if ( bSig <= aSig ) aSig -= bSig;
-        aSig64 = ( (uint64_t) aSig )<<40;
-        bSig64 = ( (uint64_t) bSig )<<40;
+        aSig64 = ( (uint64_t) aSig ) << 40;
+        bSig64 = ( (uint64_t) bSig ) << 40;
         expDiff -= 64;
         while ( 0 < expDiff ) {
             q64 = estimateDiv128To64( aSig64, 0, bSig64 );
             q64 = ( 2 < q64 ) ? q64 - 2 : 0;
-            aSig64 = - ( ( bSig * q64 )<<38 );
+            aSig64 = - ( ( bSig * q64 ) << 38 );
             expDiff -= 62;
         }
         expDiff += 64;
         q64 = estimateDiv128To64( aSig64, 0, bSig64 );
         q64 = ( 2 < q64 ) ? q64 - 2 : 0;
-        q = q64>>( 64 - expDiff );
+        q = q64 >> ( 64 - expDiff );
         bSig <<= 6;
-        aSig = ( ( aSig64>>33 )<<( expDiff - 1 ) ) - bSig * q;
+        aSig = ( ( aSig64 >> 33 ) << ( expDiff - 1 ) ) - bSig * q;
     }
     do {
         alternateASig = aSig;
@@ -5302,7 +5302,7 @@ float64 float64_rem(float64 a, float64 b, float_status *status)
     while ( 0 < expDiff ) {
         q = estimateDiv128To64( aSig, 0, bSig );
         q = ( 2 < q ) ? q - 2 : 0;
-        aSig = - ( ( bSig>>2 ) * q );
+        aSig = - ( ( bSig >> 2 ) * q );
         expDiff -= 62;
     }
     expDiff += 64;
@@ -5311,7 +5311,7 @@ float64 float64_rem(float64 a, float64 b, float_status *status)
         q = ( 2 < q ) ? q - 2 : 0;
         q >>= 64 - expDiff;
         bSig >>= 2;
-        aSig = ( ( aSig>>1 )<<( expDiff - 1 ) ) - bSig * q;
+        aSig = ( ( aSig >> 1 ) << ( expDiff - 1 ) ) - bSig * q;
     }
     else {
         aSig >>= 2;
@@ -5404,7 +5404,7 @@ int32_t floatx80_to_int32(floatx80 a, float_status *status)
     aSig = extractFloatx80Frac( a );
     aExp = extractFloatx80Exp( a );
     aSign = extractFloatx80Sign( a );
-    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig<<1 ) ) aSign = 0;
+    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;
     shiftCount = 0x4037 - aExp;
     if ( shiftCount <= 0 ) shiftCount = 1;
     shift64RightJamming( aSig, shiftCount, &aSig );
@@ -5437,7 +5437,7 @@ int32_t floatx80_to_int32_round_to_zero(floatx80 a, float_status *status)
     aExp = extractFloatx80Exp( a );
     aSign = extractFloatx80Sign( a );
     if ( 0x401E < aExp ) {
-        if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig<<1 ) ) aSign = 0;
+        if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;
         goto invalid;
     }
     else if ( aExp < 0x3FFF ) {
@@ -5456,7 +5456,7 @@ int32_t floatx80_to_int32_round_to_zero(floatx80 a, float_status *status)
         float_raise(float_flag_invalid, status);
         return aSign ? (int32_t) 0x80000000 : 0x7FFFFFFF;
     }
-    if ( ( aSig<<shiftCount ) != savedASig ) {
+    if ( ( aSig << shiftCount ) != savedASig ) {
         status->float_exception_flags |= float_flag_inexact;
     }
     return z;
@@ -5545,8 +5545,8 @@ int64_t floatx80_to_int64_round_to_zero(floatx80 a, float_status *status)
         }
         return 0;
     }
-    z = aSig>>( - shiftCount );
-    if ( (uint64_t) ( aSig<<( shiftCount & 63 ) ) ) {
+    z = aSig >> ( - shiftCount );
+    if ( (uint64_t) ( aSig << ( shiftCount & 63 ) ) ) {
         status->float_exception_flags |= float_flag_inexact;
     }
     if ( aSign ) z = - z;
@@ -5575,7 +5575,7 @@ float32 floatx80_to_float32(floatx80 a, float_status *status)
     aExp = extractFloatx80Exp( a );
     aSign = extractFloatx80Sign( a );
     if ( aExp == 0x7FFF ) {
-        if ( (uint64_t) ( aSig<<1 ) ) {
+        if ( (uint64_t) ( aSig << 1 ) ) {
             float32 res = commonNaNToFloat32(floatx80ToCommonNaN(a, status),
                                              status);
             return float32_silence_nan(res, status);
@@ -5609,7 +5609,7 @@ float64 floatx80_to_float64(floatx80 a, float_status *status)
     aExp = extractFloatx80Exp( a );
     aSign = extractFloatx80Sign( a );
     if ( aExp == 0x7FFF ) {
-        if ( (uint64_t) ( aSig<<1 ) ) {
+        if ( (uint64_t) ( aSig << 1 ) ) {
             float64 res = commonNaNToFloat64(floatx80ToCommonNaN(a, status),
                                              status);
             return float64_silence_nan(res, status);
@@ -5642,12 +5642,12 @@ float128 floatx80_to_float128(floatx80 a, float_status *status)
     aSig = extractFloatx80Frac( a );
     aExp = extractFloatx80Exp( a );
     aSign = extractFloatx80Sign( a );
-    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig<<1 ) ) {
+    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) {
         float128 res = commonNaNToFloat128(floatx80ToCommonNaN(a, status),
                                            status);
         return float128_silence_nan(res, status);
     }
-    shift128Right( aSig<<1, 0, 16, &zSig0, &zSig1 );
+    shift128Right( aSig << 1, 0, 16, &zSig0, &zSig1 );
     return packFloat128( aSign, aExp, zSig0, zSig1 );
 
 }
@@ -5688,7 +5688,7 @@ floatx80 floatx80_round_to_int(floatx80 a, float_status *status)
     }
     aExp = extractFloatx80Exp( a );
     if ( 0x403E <= aExp ) {
-        if ( ( aExp == 0x7FFF ) && (uint64_t) ( extractFloatx80Frac( a )<<1 ) ) {
+        if ( ( aExp == 0x7FFF ) && (uint64_t) ( extractFloatx80Frac( a ) << 1 ) ) {
             return propagateFloatx80NaN(a, a, status);
         }
         return a;
@@ -5702,7 +5702,7 @@ floatx80 floatx80_round_to_int(floatx80 a, float_status *status)
         aSign = extractFloatx80Sign( a );
         switch (status->float_rounding_mode) {
          case float_round_nearest_even:
-            if ( ( aExp == 0x3FFE ) && (uint64_t) ( extractFloatx80Frac( a )<<1 )
+            if ( ( aExp == 0x3FFE ) && (uint64_t) ( extractFloatx80Frac( a ) << 1 )
                ) {
                 return
                     packFloatx80( aSign, 0x3FFF, UINT64_C(0x8000000000000000));
@@ -5736,7 +5736,7 @@ floatx80 floatx80_round_to_int(floatx80 a, float_status *status)
     z = a;
     switch (status->float_rounding_mode) {
     case float_round_nearest_even:
-        z.low += lastBitMask>>1;
+        z.low += lastBitMask >> 1;
         if ((z.low & roundBitsMask) == 0) {
             z.low &= ~lastBitMask;
         }
@@ -5817,7 +5817,7 @@ static floatx80 addFloatx80Sigs(floatx80 a, floatx80 b, bool zSign,
     }
     else {
         if ( aExp == 0x7FFF ) {
-            if ( (uint64_t) ( ( aSig | bSig )<<1 ) ) {
+            if ( (uint64_t) ( ( aSig | bSig ) << 1 ) ) {
                 return propagateFloatx80NaN(a, b, status);
             }
             return a;
@@ -5874,7 +5874,7 @@ static floatx80 subFloatx80Sigs(floatx80 a, floatx80 b, bool zSign,
     if ( 0 < expDiff ) goto aExpBigger;
     if ( expDiff < 0 ) goto bExpBigger;
     if ( aExp == 0x7FFF ) {
-        if ( (uint64_t) ( ( aSig | bSig )<<1 ) ) {
+        if ( (uint64_t) ( ( aSig | bSig ) << 1 ) ) {
             return propagateFloatx80NaN(a, b, status);
         }
         float_raise(float_flag_invalid, status);
@@ -5994,8 +5994,8 @@ floatx80 floatx80_mul(floatx80 a, floatx80 b, float_status *status)
     bSign = extractFloatx80Sign( b );
     zSign = aSign ^ bSign;
     if ( aExp == 0x7FFF ) {
-        if (    (uint64_t) ( aSig<<1 )
-             || ( ( bExp == 0x7FFF ) && (uint64_t) ( bSig<<1 ) ) ) {
+        if (    (uint64_t) ( aSig << 1 )
+             || ( ( bExp == 0x7FFF ) && (uint64_t) ( bSig << 1 ) ) ) {
             return propagateFloatx80NaN(a, b, status);
         }
         if ( ( bExp | bSig ) == 0 ) goto invalid;
@@ -6106,7 +6106,7 @@ floatx80 floatx80_div(floatx80 a, floatx80 b, float_status *status)
         add128( rem0, rem1, 0, bSig, &rem0, &rem1 );
     }
     zSig1 = estimateDiv128To64( rem1, 0, bSig );
-    if ( (uint64_t) ( zSig1<<1 ) <= 8 ) {
+    if ( (uint64_t) ( zSig1 << 1 ) <= 8 ) {
         mul64To128( bSig, zSig1, &term1, &term2 );
         sub128( rem1, 0, term1, term2, &rem1, &rem2 );
         while ( (int64_t) rem1 < 0 ) {
@@ -6147,8 +6147,8 @@ floatx80 floatx80_modrem(floatx80 a, floatx80 b, bool mod, uint64_t *quotient,
     bSig = extractFloatx80Frac( b );
     bExp = extractFloatx80Exp( b );
     if ( aExp == 0x7FFF ) {
-        if (    (uint64_t) ( aSig0<<1 )
-             || ( ( bExp == 0x7FFF ) && (uint64_t) ( bSig<<1 ) ) ) {
+        if (    (uint64_t) ( aSig0 << 1 )
+             || ( ( bExp == 0x7FFF ) && (uint64_t) ( bSig << 1 ) ) ) {
             return propagateFloatx80NaN(a, b, status);
         }
         goto invalid;
@@ -6213,7 +6213,7 @@ floatx80 floatx80_modrem(floatx80 a, floatx80 b, bool mod, uint64_t *quotient,
         q = estimateDiv128To64( aSig0, aSig1, bSig );
         q = ( 2 < q ) ? q - 2 : 0;
         q >>= 64 - expDiff;
-        mul64To128( bSig, q<<( 64 - expDiff ), &term0, &term1 );
+        mul64To128( bSig, q << ( 64 - expDiff ), &term0, &term1 );
         sub128( aSig0, aSig1, term0, term1, &aSig0, &aSig1 );
         shortShift128Left( 0, bSig, 64 - expDiff, &term0, &term1 );
         while ( le128( term0, term1, aSig0, aSig1 ) ) {
@@ -6310,17 +6310,17 @@ floatx80 floatx80_sqrt(floatx80 a, float_status *status)
         if ( aSig0 == 0 ) return packFloatx80( 0, 0, 0 );
         normalizeFloatx80Subnormal( aSig0, &aExp, &aSig0 );
     }
-    zExp = ( ( aExp - 0x3FFF )>>1 ) + 0x3FFF;
-    zSig0 = estimateSqrt32( aExp, aSig0>>32 );
+    zExp = ( ( aExp - 0x3FFF ) >> 1 ) + 0x3FFF;
+    zSig0 = estimateSqrt32( aExp, aSig0 >> 32 );
     shift128Right( aSig0, 0, 2 + ( aExp & 1 ), &aSig0, &aSig1 );
-    zSig0 = estimateDiv128To64( aSig0, aSig1, zSig0<<32 ) + ( zSig0<<30 );
-    doubleZSig0 = zSig0<<1;
+    zSig0 = estimateDiv128To64( aSig0, aSig1, zSig0 << 32 ) + ( zSig0 << 30 );
+    doubleZSig0 = zSig0 << 1;
     mul64To128( zSig0, zSig0, &term0, &term1 );
     sub128( aSig0, aSig1, term0, term1, &rem0, &rem1 );
     while ( (int64_t) rem0 < 0 ) {
         --zSig0;
         doubleZSig0 -= 2;
-        add128( rem0, rem1, zSig0>>63, doubleZSig0 | 1, &rem0, &rem1 );
+        add128( rem0, rem1, zSig0 >> 63, doubleZSig0 | 1, &rem0, &rem1 );
     }
     zSig1 = estimateDiv128To64( rem1, 0, doubleZSig0 );
     if ( ( zSig1 & UINT64_C(0x3FFFFFFFFFFFFFFF) ) <= 5 ) {
@@ -6416,7 +6416,7 @@ int32_t float128_to_int32_round_to_zero(float128 a, float_status *status)
         float_raise(float_flag_invalid, status);
         return aSign ? INT32_MIN : INT32_MAX;
     }
-    if ( ( aSig0<<shiftCount ) != savedASig ) {
+    if ( ( aSig0 << shiftCount ) != savedASig ) {
         status->float_exception_flags |= float_flag_inexact;
     }
     return z;
@@ -6506,8 +6506,8 @@ int64_t float128_to_int64_round_to_zero(float128 a, float_status *status)
             }
             return INT64_MIN;
         }
-        z = ( aSig0<<shiftCount ) | ( aSig1>>( ( - shiftCount ) & 63 ) );
-        if ( (uint64_t) ( aSig1<<shiftCount ) ) {
+        z = ( aSig0 << shiftCount ) | ( aSig1 >> ( ( - shiftCount ) & 63 ) );
+        if ( (uint64_t) ( aSig1 << shiftCount ) ) {
             status->float_exception_flags |= float_flag_inexact;
         }
     }
@@ -6518,9 +6518,9 @@ int64_t float128_to_int64_round_to_zero(float128 a, float_status *status)
             }
             return 0;
         }
-        z = aSig0>>( - shiftCount );
+        z = aSig0 >> ( - shiftCount );
         if (    aSig1
-             || ( shiftCount && (uint64_t) ( aSig0<<( shiftCount & 63 ) ) ) ) {
+             || ( shiftCount && (uint64_t) ( aSig0 << ( shiftCount & 63 ) ) ) ) {
             status->float_exception_flags |= float_flag_inexact;
         }
     }
@@ -6776,19 +6776,19 @@ float128 float128_round_to_int(float128 a, float_status *status)
             return a;
         }
         lastBitMask = 1;
-        lastBitMask = ( lastBitMask<<( 0x406E - aExp ) )<<1;
+        lastBitMask = ( lastBitMask << ( 0x406E - aExp ) ) << 1;
         roundBitsMask = lastBitMask - 1;
         z = a;
         switch (status->float_rounding_mode) {
         case float_round_nearest_even:
             if ( lastBitMask ) {
-                add128( z.high, z.low, 0, lastBitMask>>1, &z.high, &z.low );
+                add128( z.high, z.low, 0, lastBitMask >> 1, &z.high, &z.low );
                 if ( ( z.low & roundBitsMask ) == 0 ) z.low &= ~ lastBitMask;
             }
             else {
                 if ( (int64_t) z.low < 0 ) {
                     ++z.high;
-                    if ( (uint64_t) ( z.low<<1 ) == 0 ) z.high &= ~1;
+                    if ( (uint64_t) ( z.low << 1 ) == 0 ) z.high &= ~1;
                 }
             }
             break;
@@ -6829,7 +6829,7 @@ float128 float128_round_to_int(float128 a, float_status *status)
     }
     else {
         if ( aExp < 0x3FFF ) {
-            if ( ( ( (uint64_t) ( a.high<<1 ) ) | a.low ) == 0 ) return a;
+            if ( ( ( (uint64_t) ( a.high << 1 ) ) | a.low ) == 0 ) return a;
             status->float_exception_flags |= float_flag_inexact;
             aSign = extractFloat128Sign( a );
             switch (status->float_rounding_mode) {
@@ -6870,13 +6870,13 @@ float128 float128_round_to_int(float128 a, float_status *status)
         z.high = a.high;
         switch (status->float_rounding_mode) {
         case float_round_nearest_even:
-            z.high += lastBitMask>>1;
+            z.high += lastBitMask >> 1;
             if ( ( ( z.high & roundBitsMask ) | a.low ) == 0 ) {
                 z.high &= ~ lastBitMask;
             }
             break;
         case float_round_ties_away:
-            z.high += lastBitMask>>1;
+            z.high += lastBitMask >> 1;
             break;
         case float_round_to_zero:
             break;
@@ -7422,18 +7422,18 @@ float128 float128_sqrt(float128 a, float_status *status)
         if ( ( aSig0 | aSig1 ) == 0 ) return packFloat128( 0, 0, 0, 0 );
         normalizeFloat128Subnormal( aSig0, aSig1, &aExp, &aSig0, &aSig1 );
     }
-    zExp = ( ( aExp - 0x3FFF )>>1 ) + 0x3FFE;
+    zExp = ( ( aExp - 0x3FFF ) >> 1 ) + 0x3FFE;
     aSig0 |= UINT64_C(0x0001000000000000);
-    zSig0 = estimateSqrt32( aExp, aSig0>>17 );
+    zSig0 = estimateSqrt32( aExp, aSig0 >> 17 );
     shortShift128Left( aSig0, aSig1, 13 - ( aExp & 1 ), &aSig0, &aSig1 );
-    zSig0 = estimateDiv128To64( aSig0, aSig1, zSig0<<32 ) + ( zSig0<<30 );
-    doubleZSig0 = zSig0<<1;
+    zSig0 = estimateDiv128To64( aSig0, aSig1, zSig0 << 32 ) + ( zSig0 << 30 );
+    doubleZSig0 = zSig0 << 1;
     mul64To128( zSig0, zSig0, &term0, &term1 );
     sub128( aSig0, aSig1, term0, term1, &rem0, &rem1 );
     while ( (int64_t) rem0 < 0 ) {
         --zSig0;
         doubleZSig0 -= 2;
-        add128( rem0, rem1, zSig0>>63, doubleZSig0 | 1, &rem0, &rem1 );
+        add128( rem0, rem1, zSig0 >> 63, doubleZSig0 | 1, &rem0, &rem1 );
     }
     zSig1 = estimateDiv128To64( rem1, 0, doubleZSig0 );
     if ( ( zSig1 & 0x1FFF ) <= 5 ) {
@@ -7467,9 +7467,9 @@ floatx80_compare_internal(floatx80 a, floatx80 b, bool is_quiet,
         return float_relation_unordered;
     }
     if (( ( extractFloatx80Exp( a ) == 0x7fff ) &&
-          ( extractFloatx80Frac( a )<<1 ) ) ||
+          ( extractFloatx80Frac( a ) << 1 ) ) ||
         ( ( extractFloatx80Exp( b ) == 0x7fff ) &&
-          ( extractFloatx80Frac( b )<<1 ) )) {
+          ( extractFloatx80Frac( b ) << 1 ) )) {
         if (!is_quiet ||
             floatx80_is_signaling_nan(a, status) ||
             floatx80_is_signaling_nan(b, status)) {
@@ -7535,7 +7535,7 @@ float128_compare_internal(float128 a, float128 b, bool is_quiet,
     aSign = extractFloat128Sign( a );
     bSign = extractFloat128Sign( b );
     if ( aSign != bSign ) {
-        if ( ( ( ( a.high | b.high )<<1 ) | a.low | b.low ) == 0 ) {
+        if ( ( ( ( a.high | b.high ) << 1 ) | a.low | b.low ) == 0 ) {
             /* zero case */
             return float_relation_equal;
         } else {
@@ -7576,7 +7576,7 @@ floatx80 floatx80_scalbn(floatx80 a, int n, float_status *status)
     aSign = extractFloatx80Sign( a );
 
     if ( aExp == 0x7FFF ) {
-        if ( aSig<<1 ) {
+        if ( aSig << 1 ) {
             return propagateFloatx80NaN(a, a, status);
         }
         return a;
-- 
2.29.0-rc1


Re: [PATCH] fpu/softfloat.c: add spaces around operator
Posted by no-reply@patchew.org 3 years, 5 months ago
Patchew URL: https://patchew.org/QEMU/20201114130715.1126922-1-zhangxinhao1@huawei.com/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Message-id: 20201114130715.1126922-1-zhangxinhao1@huawei.com
Type: series
Subject: [PATCH] fpu/softfloat.c: add spaces around operator

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
 - [tag update]      patchew/20201113183312.432630-1-michael.roth@amd.com -> patchew/20201113183312.432630-1-michael.roth@amd.com
 * [new tag]         patchew/20201114130715.1126922-1-zhangxinhao1@huawei.com -> patchew/20201114130715.1126922-1-zhangxinhao1@huawei.com
Switched to a new branch 'test'
16278b7 fpu/softfloat.c: add spaces around operator

=== OUTPUT BEGIN ===
ERROR: space prohibited after that open parenthesis '('
#22: FILE: fpu/softfloat.c:3789:
+    absZ = ( absZ + roundIncrement ) >> 7;

ERROR: space prohibited before that close parenthesis ')'
#22: FILE: fpu/softfloat.c:3789:
+    absZ = ( absZ + roundIncrement ) >> 7;

ERROR: space prohibited after that open parenthesis '('
#29: FILE: fpu/softfloat.c:3795:
+    if ( ( absZ >> 32 ) || ( z && ( ( z < 0 ) ^ zSign ) ) ) {

ERROR: space prohibited before that close parenthesis ')'
#29: FILE: fpu/softfloat.c:3795:
+    if ( ( absZ >> 32 ) || ( z && ( ( z < 0 ) ^ zSign ) ) ) {

ERROR: space prohibited after that open parenthesis '('
#47: FILE: fpu/softfloat.c:4036:
+    zSig = ( zSig + roundIncrement ) >> 7;

ERROR: space prohibited before that close parenthesis ')'
#47: FILE: fpu/softfloat.c:4036:
+    zSig = ( zSig + roundIncrement ) >> 7;

ERROR: space prohibited after that open parenthesis '('
#74: FILE: fpu/softfloat.c:4099:
+        ( ( (uint64_t) zSign ) << 63 ) + ( ( (uint64_t) zExp ) << 52 ) + zSig);

ERROR: space prohibited before that close parenthesis ')'
#74: FILE: fpu/softfloat.c:4099:
+        ( ( (uint64_t) zSign ) << 63 ) + ( ( (uint64_t) zExp ) << 52 ) + zSig);

ERROR: space prohibited after that open parenthesis '('
#83: FILE: fpu/softfloat.c:4192:
+    zSig = ( zSig + roundIncrement ) >> 10;

ERROR: space prohibited before that close parenthesis ')'
#83: FILE: fpu/softfloat.c:4192:
+    zSig = ( zSig + roundIncrement ) >> 10;

ERROR: space prohibited after that open parenthesis '('
#110: FILE: fpu/softfloat.c:4329:
+            if ( roundNearestEven && ( roundBits << 1 == roundIncrement ) ) {

ERROR: space prohibited before that close parenthesis ')'
#110: FILE: fpu/softfloat.c:4329:
+            if ( roundNearestEven && ( roundBits << 1 == roundIncrement ) ) {

ERROR: space prohibited after that open parenthesis '('
#119: FILE: fpu/softfloat.c:4345:
+    if ( roundNearestEven && ( roundBits << 1 == roundIncrement ) ) {

ERROR: space prohibited before that close parenthesis ')'
#119: FILE: fpu/softfloat.c:4345:
+    if ( roundNearestEven && ( roundBits << 1 == roundIncrement ) ) {

ERROR: space prohibited after that open parenthesis '('
#128: FILE: fpu/softfloat.c:4512:
+    return ( a.high >> 48 ) & 0x7FFF;

ERROR: space prohibited before that close parenthesis ')'
#128: FILE: fpu/softfloat.c:4512:
+    return ( a.high >> 48 ) & 0x7FFF;

ERROR: space prohibited after that '-' (ctx:WxW)
#138: FILE: fpu/softfloat.c:4549:
+            *zSig0Ptr = aSig1 >> ( - shiftCount );
                                    ^

ERROR: space prohibited after that open parenthesis '('
#138: FILE: fpu/softfloat.c:4549:
+            *zSig0Ptr = aSig1 >> ( - shiftCount );

ERROR: space prohibited before that close parenthesis ')'
#138: FILE: fpu/softfloat.c:4549:
+            *zSig0Ptr = aSig1 >> ( - shiftCount );

ERROR: space prohibited after that open parenthesis '('
#139: FILE: fpu/softfloat.c:4550:
+            *zSig1Ptr = aSig1 << ( shiftCount & 63 );

ERROR: space prohibited before that close parenthesis ')'
#139: FILE: fpu/softfloat.c:4550:
+            *zSig1Ptr = aSig1 << ( shiftCount & 63 );

ERROR: space prohibited after that open parenthesis '('
#152: FILE: fpu/softfloat.c:4778:
+    return packFloatx80( zSign, 0x403E - shiftCount, zSig << shiftCount );

ERROR: space prohibited before that close parenthesis ')'
#152: FILE: fpu/softfloat.c:4778:
+    return packFloatx80( zSign, 0x403E - shiftCount, zSig << shiftCount );

ERROR: space prohibited after that open parenthesis '('
#161: FILE: fpu/softfloat.c:4800:
+    return packFloat128( zSign, 0x402E - shiftCount, zSig0 << shiftCount, 0 );

ERROR: space prohibited before that close parenthesis ')'
#161: FILE: fpu/softfloat.c:4800:
+    return packFloat128( zSign, 0x402E - shiftCount, zSig0 << shiftCount, 0 );

ERROR: space prohibited after that open parenthesis '('
#170: FILE: fpu/softfloat.c:4821:
+    return packFloatx80( zSign, 0x403E - shiftCount, absA << shiftCount );

ERROR: space prohibited before that close parenthesis ')'
#170: FILE: fpu/softfloat.c:4821:
+    return packFloatx80( zSign, 0x403E - shiftCount, absA << shiftCount );

ERROR: space prohibited after that open parenthesis '('
#179: FILE: fpu/softfloat.c:4904:
+    return packFloatx80( aSign, aExp + 0x3F80, ( (uint64_t) aSig ) << 40 );

ERROR: space prohibited before that close parenthesis ')'
#179: FILE: fpu/softfloat.c:4904:
+    return packFloatx80( aSign, aExp + 0x3F80, ( (uint64_t) aSig ) << 40 );

ERROR: space prohibited after that open parenthesis '('
#188: FILE: fpu/softfloat.c:4936:
+    return packFloat128( aSign, aExp + 0x3F80, ( (uint64_t) aSig ) << 25, 0 );

ERROR: space prohibited before that close parenthesis ')'
#188: FILE: fpu/softfloat.c:4936:
+    return packFloat128( aSign, aExp + 0x3F80, ( (uint64_t) aSig ) << 25, 0 );

ERROR: space prohibited after that open parenthesis '('
#197: FILE: fpu/softfloat.c:5000:
+            q = ( ( (uint64_t) aSig ) << 32 ) / bSig;

ERROR: space prohibited before that close parenthesis ')'
#197: FILE: fpu/softfloat.c:5000:
+            q = ( ( (uint64_t) aSig ) << 32 ) / bSig;

ERROR: space prohibited after that open parenthesis '('
#201: FILE: fpu/softfloat.c:5003:
+            aSig = ( ( aSig >> 1 ) << ( expDiff - 1 ) ) - bSig * q;

ERROR: space prohibited before that close parenthesis ')'
#201: FILE: fpu/softfloat.c:5003:
+            aSig = ( ( aSig >> 1 ) << ( expDiff - 1 ) ) - bSig * q;

ERROR: space prohibited after that open parenthesis '('
#211: FILE: fpu/softfloat.c:5012:
+        aSig64 = ( (uint64_t) aSig ) << 40;

ERROR: space prohibited before that close parenthesis ')'
#211: FILE: fpu/softfloat.c:5012:
+        aSig64 = ( (uint64_t) aSig ) << 40;

ERROR: space prohibited after that open parenthesis '('
#212: FILE: fpu/softfloat.c:5013:
+        bSig64 = ( (uint64_t) bSig ) << 40;

ERROR: space prohibited before that close parenthesis ')'
#212: FILE: fpu/softfloat.c:5013:
+        bSig64 = ( (uint64_t) bSig ) << 40;

ERROR: space prohibited after that '-' (ctx:WxW)
#218: FILE: fpu/softfloat.c:5018:
+            aSig64 = - ( ( bSig * q64 ) << 38 );
                      ^

ERROR: space prohibited after that open parenthesis '('
#218: FILE: fpu/softfloat.c:5018:
+            aSig64 = - ( ( bSig * q64 ) << 38 );

ERROR: space prohibited before that close parenthesis ')'
#218: FILE: fpu/softfloat.c:5018:
+            aSig64 = - ( ( bSig * q64 ) << 38 );

ERROR: space prohibited after that open parenthesis '('
#225: FILE: fpu/softfloat.c:5024:
+        q = q64 >> ( 64 - expDiff );

ERROR: space prohibited before that close parenthesis ')'
#225: FILE: fpu/softfloat.c:5024:
+        q = q64 >> ( 64 - expDiff );

ERROR: space prohibited after that open parenthesis '('
#228: FILE: fpu/softfloat.c:5026:
+        aSig = ( ( aSig64 >> 33 ) << ( expDiff - 1 ) ) - bSig * q;

ERROR: space prohibited before that close parenthesis ')'
#228: FILE: fpu/softfloat.c:5026:
+        aSig = ( ( aSig64 >> 33 ) << ( expDiff - 1 ) ) - bSig * q;

ERROR: space prohibited after that '-' (ctx:WxW)
#237: FILE: fpu/softfloat.c:5305:
+        aSig = - ( ( bSig >> 2 ) * q );
                ^

ERROR: space prohibited after that open parenthesis '('
#237: FILE: fpu/softfloat.c:5305:
+        aSig = - ( ( bSig >> 2 ) * q );

ERROR: space prohibited before that close parenthesis ')'
#237: FILE: fpu/softfloat.c:5305:
+        aSig = - ( ( bSig >> 2 ) * q );

ERROR: space prohibited after that open parenthesis '('
#246: FILE: fpu/softfloat.c:5314:
+        aSig = ( ( aSig >> 1 ) << ( expDiff - 1 ) ) - bSig * q;

ERROR: space prohibited before that close parenthesis ')'
#246: FILE: fpu/softfloat.c:5314:
+        aSig = ( ( aSig >> 1 ) << ( expDiff - 1 ) ) - bSig * q;

ERROR: space prohibited after that open parenthesis '('
#255: FILE: fpu/softfloat.c:5407:
+    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;

ERROR: space prohibited before that close parenthesis ')'
#255: FILE: fpu/softfloat.c:5407:
+    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;

ERROR: trailing statements should be on next line
#255: FILE: fpu/softfloat.c:5407:
+    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;

ERROR: braces {} are necessary for all arms of this statement
#255: FILE: fpu/softfloat.c:5407:
+    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;
[...]

ERROR: space prohibited after that open parenthesis '('
#264: FILE: fpu/softfloat.c:5440:
+        if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;

ERROR: space prohibited before that close parenthesis ')'
#264: FILE: fpu/softfloat.c:5440:
+        if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;

ERROR: trailing statements should be on next line
#264: FILE: fpu/softfloat.c:5440:
+        if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;

ERROR: braces {} are necessary for all arms of this statement
#264: FILE: fpu/softfloat.c:5440:
+        if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) aSign = 0;
[...]

ERROR: space prohibited after that open parenthesis '('
#273: FILE: fpu/softfloat.c:5459:
+    if ( ( aSig << shiftCount ) != savedASig ) {

ERROR: space prohibited before that close parenthesis ')'
#273: FILE: fpu/softfloat.c:5459:
+    if ( ( aSig << shiftCount ) != savedASig ) {

ERROR: space prohibited after that '-' (ctx:WxW)
#283: FILE: fpu/softfloat.c:5548:
+    z = aSig >> ( - shiftCount );
                   ^

ERROR: space prohibited after that open parenthesis '('
#283: FILE: fpu/softfloat.c:5548:
+    z = aSig >> ( - shiftCount );

ERROR: space prohibited before that close parenthesis ')'
#283: FILE: fpu/softfloat.c:5548:
+    z = aSig >> ( - shiftCount );

ERROR: space prohibited after that open parenthesis '('
#284: FILE: fpu/softfloat.c:5549:
+    if ( (uint64_t) ( aSig << ( shiftCount & 63 ) ) ) {

ERROR: space prohibited before that close parenthesis ')'
#284: FILE: fpu/softfloat.c:5549:
+    if ( (uint64_t) ( aSig << ( shiftCount & 63 ) ) ) {

ERROR: space prohibited after that open parenthesis '('
#293: FILE: fpu/softfloat.c:5578:
+        if ( (uint64_t) ( aSig << 1 ) ) {

ERROR: space prohibited before that close parenthesis ')'
#293: FILE: fpu/softfloat.c:5578:
+        if ( (uint64_t) ( aSig << 1 ) ) {

ERROR: space prohibited after that open parenthesis '('
#302: FILE: fpu/softfloat.c:5612:
+        if ( (uint64_t) ( aSig << 1 ) ) {

ERROR: space prohibited before that close parenthesis ')'
#302: FILE: fpu/softfloat.c:5612:
+        if ( (uint64_t) ( aSig << 1 ) ) {

ERROR: space prohibited after that open parenthesis '('
#311: FILE: fpu/softfloat.c:5645:
+    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) {

ERROR: space prohibited before that close parenthesis ')'
#311: FILE: fpu/softfloat.c:5645:
+    if ( ( aExp == 0x7FFF ) && (uint64_t) ( aSig << 1 ) ) {

ERROR: space prohibited after that open parenthesis '('
#317: FILE: fpu/softfloat.c:5650:
+    shift128Right( aSig << 1, 0, 16, &zSig0, &zSig1 );

ERROR: space prohibited before that close parenthesis ')'
#317: FILE: fpu/softfloat.c:5650:
+    shift128Right( aSig << 1, 0, 16, &zSig0, &zSig1 );

WARNING: line over 80 characters
#326: FILE: fpu/softfloat.c:5691:
+        if ( ( aExp == 0x7FFF ) && (uint64_t) ( extractFloatx80Frac( a ) << 1 ) ) {

ERROR: space prohibited after that open parenthesis '('
#326: FILE: fpu/softfloat.c:5691:
+        if ( ( aExp == 0x7FFF ) && (uint64_t) ( extractFloatx80Frac( a ) << 1 ) ) {

ERROR: space prohibited before that close parenthesis ')'
#326: FILE: fpu/softfloat.c:5691:
+        if ( ( aExp == 0x7FFF ) && (uint64_t) ( extractFloatx80Frac( a ) << 1 ) ) {

WARNING: line over 80 characters
#335: FILE: fpu/softfloat.c:5705:
+            if ( ( aExp == 0x3FFE ) && (uint64_t) ( extractFloatx80Frac( a ) << 1 )

ERROR: space prohibited after that open parenthesis '('
#335: FILE: fpu/softfloat.c:5705:
+            if ( ( aExp == 0x3FFE ) && (uint64_t) ( extractFloatx80Frac( a ) << 1 )

ERROR: space prohibited before that close parenthesis ')'
#335: FILE: fpu/softfloat.c:5705:
+            if ( ( aExp == 0x3FFE ) && (uint64_t) ( extractFloatx80Frac( a ) << 1 )

ERROR: space prohibited after that open parenthesis '('
#353: FILE: fpu/softfloat.c:5820:
+            if ( (uint64_t) ( ( aSig | bSig ) << 1 ) ) {

ERROR: space prohibited before that close parenthesis ')'
#353: FILE: fpu/softfloat.c:5820:
+            if ( (uint64_t) ( ( aSig | bSig ) << 1 ) ) {

ERROR: space prohibited after that open parenthesis '('
#362: FILE: fpu/softfloat.c:5877:
+        if ( (uint64_t) ( ( aSig | bSig ) << 1 ) ) {

ERROR: space prohibited before that close parenthesis ')'
#362: FILE: fpu/softfloat.c:5877:
+        if ( (uint64_t) ( ( aSig | bSig ) << 1 ) ) {

ERROR: space prohibited after that open parenthesis '('
#372: FILE: fpu/softfloat.c:5997:
+        if (    (uint64_t) ( aSig << 1 )

ERROR: space prohibited before that close parenthesis ')'
#372: FILE: fpu/softfloat.c:5997:
+        if (    (uint64_t) ( aSig << 1 )

ERROR: space prohibited after that open parenthesis '('
#373: FILE: fpu/softfloat.c:5998:
+             || ( ( bExp == 0x7FFF ) && (uint64_t) ( bSig << 1 ) ) ) {

ERROR: space prohibited before that close parenthesis ')'
#373: FILE: fpu/softfloat.c:5998:
+             || ( ( bExp == 0x7FFF ) && (uint64_t) ( bSig << 1 ) ) ) {

ERROR: space prohibited after that open parenthesis '('
#382: FILE: fpu/softfloat.c:6109:
+    if ( (uint64_t) ( zSig1 << 1 ) <= 8 ) {

ERROR: space prohibited before that close parenthesis ')'
#382: FILE: fpu/softfloat.c:6109:
+    if ( (uint64_t) ( zSig1 << 1 ) <= 8 ) {

ERROR: space prohibited after that open parenthesis '('
#392: FILE: fpu/softfloat.c:6150:
+        if (    (uint64_t) ( aSig0 << 1 )

ERROR: space prohibited before that close parenthesis ')'
#392: FILE: fpu/softfloat.c:6150:
+        if (    (uint64_t) ( aSig0 << 1 )

ERROR: space prohibited after that open parenthesis '('
#393: FILE: fpu/softfloat.c:6151:
+             || ( ( bExp == 0x7FFF ) && (uint64_t) ( bSig << 1 ) ) ) {

ERROR: space prohibited before that close parenthesis ')'
#393: FILE: fpu/softfloat.c:6151:
+             || ( ( bExp == 0x7FFF ) && (uint64_t) ( bSig << 1 ) ) ) {

ERROR: space prohibited after that open parenthesis '('
#402: FILE: fpu/softfloat.c:6216:
+        mul64To128( bSig, q << ( 64 - expDiff ), &term0, &term1 );

ERROR: space prohibited before that close parenthesis ')'
#402: FILE: fpu/softfloat.c:6216:
+        mul64To128( bSig, q << ( 64 - expDiff ), &term0, &term1 );

ERROR: space prohibited after that open parenthesis '('
#412: FILE: fpu/softfloat.c:6313:
+    zExp = ( ( aExp - 0x3FFF ) >> 1 ) + 0x3FFF;

ERROR: space prohibited before that close parenthesis ')'
#412: FILE: fpu/softfloat.c:6313:
+    zExp = ( ( aExp - 0x3FFF ) >> 1 ) + 0x3FFF;

ERROR: space prohibited after that open parenthesis '('
#413: FILE: fpu/softfloat.c:6314:
+    zSig0 = estimateSqrt32( aExp, aSig0 >> 32 );

ERROR: space prohibited before that close parenthesis ')'
#413: FILE: fpu/softfloat.c:6314:
+    zSig0 = estimateSqrt32( aExp, aSig0 >> 32 );

ERROR: space prohibited after that open parenthesis '('
#417: FILE: fpu/softfloat.c:6316:
+    zSig0 = estimateDiv128To64( aSig0, aSig1, zSig0 << 32 ) + ( zSig0 << 30 );

ERROR: space prohibited before that close parenthesis ')'
#417: FILE: fpu/softfloat.c:6316:
+    zSig0 = estimateDiv128To64( aSig0, aSig1, zSig0 << 32 ) + ( zSig0 << 30 );

ERROR: space prohibited after that open parenthesis '('
#425: FILE: fpu/softfloat.c:6323:
+        add128( rem0, rem1, zSig0 >> 63, doubleZSig0 | 1, &rem0, &rem1 );

ERROR: space prohibited before that close parenthesis ')'
#425: FILE: fpu/softfloat.c:6323:
+        add128( rem0, rem1, zSig0 >> 63, doubleZSig0 | 1, &rem0, &rem1 );

ERROR: space prohibited after that open parenthesis '('
#434: FILE: fpu/softfloat.c:6419:
+    if ( ( aSig0 << shiftCount ) != savedASig ) {

ERROR: space prohibited before that close parenthesis ')'
#434: FILE: fpu/softfloat.c:6419:
+    if ( ( aSig0 << shiftCount ) != savedASig ) {

ERROR: space prohibited after that '-' (ctx:WxW)
#444: FILE: fpu/softfloat.c:6509:
+        z = ( aSig0 << shiftCount ) | ( aSig1 >> ( ( - shiftCount ) & 63 ) );
                                                      ^

ERROR: space prohibited after that open parenthesis '('
#444: FILE: fpu/softfloat.c:6509:
+        z = ( aSig0 << shiftCount ) | ( aSig1 >> ( ( - shiftCount ) & 63 ) );

ERROR: space prohibited before that close parenthesis ')'
#444: FILE: fpu/softfloat.c:6509:
+        z = ( aSig0 << shiftCount ) | ( aSig1 >> ( ( - shiftCount ) & 63 ) );

ERROR: space prohibited after that open parenthesis '('
#445: FILE: fpu/softfloat.c:6510:
+        if ( (uint64_t) ( aSig1 << shiftCount ) ) {

ERROR: space prohibited before that close parenthesis ')'
#445: FILE: fpu/softfloat.c:6510:
+        if ( (uint64_t) ( aSig1 << shiftCount ) ) {

ERROR: space prohibited after that '-' (ctx:WxW)
#454: FILE: fpu/softfloat.c:6521:
+        z = aSig0 >> ( - shiftCount );
                        ^

ERROR: space prohibited after that open parenthesis '('
#454: FILE: fpu/softfloat.c:6521:
+        z = aSig0 >> ( - shiftCount );

ERROR: space prohibited before that close parenthesis ')'
#454: FILE: fpu/softfloat.c:6521:
+        z = aSig0 >> ( - shiftCount );

WARNING: line over 80 characters
#457: FILE: fpu/softfloat.c:6523:
+             || ( shiftCount && (uint64_t) ( aSig0 << ( shiftCount & 63 ) ) ) ) {

ERROR: space prohibited after that open parenthesis '('
#457: FILE: fpu/softfloat.c:6523:
+             || ( shiftCount && (uint64_t) ( aSig0 << ( shiftCount & 63 ) ) ) ) {

ERROR: space prohibited before that close parenthesis ')'
#457: FILE: fpu/softfloat.c:6523:
+             || ( shiftCount && (uint64_t) ( aSig0 << ( shiftCount & 63 ) ) ) ) {

ERROR: space prohibited after that open parenthesis '('
#466: FILE: fpu/softfloat.c:6779:
+        lastBitMask = ( lastBitMask << ( 0x406E - aExp ) ) << 1;

ERROR: space prohibited before that close parenthesis ')'
#466: FILE: fpu/softfloat.c:6779:
+        lastBitMask = ( lastBitMask << ( 0x406E - aExp ) ) << 1;

ERROR: space prohibited after that open parenthesis '('
#473: FILE: fpu/softfloat.c:6785:
+                add128( z.high, z.low, 0, lastBitMask >> 1, &z.high, &z.low );

ERROR: space prohibited before that close parenthesis ')'
#473: FILE: fpu/softfloat.c:6785:
+                add128( z.high, z.low, 0, lastBitMask >> 1, &z.high, &z.low );

ERROR: space prohibited after that open parenthesis '('
#480: FILE: fpu/softfloat.c:6791:
+                    if ( (uint64_t) ( z.low << 1 ) == 0 ) z.high &= ~1;

ERROR: space prohibited before that close parenthesis ')'
#480: FILE: fpu/softfloat.c:6791:
+                    if ( (uint64_t) ( z.low << 1 ) == 0 ) z.high &= ~1;

ERROR: trailing statements should be on next line
#480: FILE: fpu/softfloat.c:6791:
+                    if ( (uint64_t) ( z.low << 1 ) == 0 ) z.high &= ~1;

ERROR: braces {} are necessary for all arms of this statement
#480: FILE: fpu/softfloat.c:6791:
+                    if ( (uint64_t) ( z.low << 1 ) == 0 ) z.high &= ~1;
[...]

ERROR: space prohibited after that open parenthesis '('
#489: FILE: fpu/softfloat.c:6832:
+            if ( ( ( (uint64_t) ( a.high << 1 ) ) | a.low ) == 0 ) return a;

ERROR: space prohibited before that close parenthesis ')'
#489: FILE: fpu/softfloat.c:6832:
+            if ( ( ( (uint64_t) ( a.high << 1 ) ) | a.low ) == 0 ) return a;

ERROR: trailing statements should be on next line
#489: FILE: fpu/softfloat.c:6832:
+            if ( ( ( (uint64_t) ( a.high << 1 ) ) | a.low ) == 0 ) return a;

ERROR: braces {} are necessary for all arms of this statement
#489: FILE: fpu/softfloat.c:6832:
+            if ( ( ( (uint64_t) ( a.high << 1 ) ) | a.low ) == 0 ) return a;
[...]

ERROR: space prohibited after that open parenthesis '('
#514: FILE: fpu/softfloat.c:7425:
+    zExp = ( ( aExp - 0x3FFF ) >> 1 ) + 0x3FFE;

ERROR: space prohibited before that close parenthesis ')'
#514: FILE: fpu/softfloat.c:7425:
+    zExp = ( ( aExp - 0x3FFF ) >> 1 ) + 0x3FFE;

ERROR: space prohibited after that open parenthesis '('
#517: FILE: fpu/softfloat.c:7427:
+    zSig0 = estimateSqrt32( aExp, aSig0 >> 17 );

ERROR: space prohibited before that close parenthesis ')'
#517: FILE: fpu/softfloat.c:7427:
+    zSig0 = estimateSqrt32( aExp, aSig0 >> 17 );

ERROR: space prohibited after that open parenthesis '('
#521: FILE: fpu/softfloat.c:7429:
+    zSig0 = estimateDiv128To64( aSig0, aSig1, zSig0 << 32 ) + ( zSig0 << 30 );

ERROR: space prohibited before that close parenthesis ')'
#521: FILE: fpu/softfloat.c:7429:
+    zSig0 = estimateDiv128To64( aSig0, aSig1, zSig0 << 32 ) + ( zSig0 << 30 );

ERROR: space prohibited after that open parenthesis '('
#529: FILE: fpu/softfloat.c:7436:
+        add128( rem0, rem1, zSig0 >> 63, doubleZSig0 | 1, &rem0, &rem1 );

ERROR: space prohibited before that close parenthesis ')'
#529: FILE: fpu/softfloat.c:7436:
+        add128( rem0, rem1, zSig0 >> 63, doubleZSig0 | 1, &rem0, &rem1 );

ERROR: space prohibited after that open parenthesis '('
#538: FILE: fpu/softfloat.c:7470:
+          ( extractFloatx80Frac( a ) << 1 ) ) ||

ERROR: space prohibited before that close parenthesis ')'
#538: FILE: fpu/softfloat.c:7470:
+          ( extractFloatx80Frac( a ) << 1 ) ) ||

ERROR: space prohibited after that open parenthesis '('
#541: FILE: fpu/softfloat.c:7472:
+          ( extractFloatx80Frac( b ) << 1 ) )) {

ERROR: space prohibited before that close parenthesis ')'
#541: FILE: fpu/softfloat.c:7472:
+          ( extractFloatx80Frac( b ) << 1 ) )) {

ERROR: space prohibited after that open parenthesis '('
#550: FILE: fpu/softfloat.c:7538:
+        if ( ( ( ( a.high | b.high ) << 1 ) | a.low | b.low ) == 0 ) {

ERROR: space prohibited before that close parenthesis ')'
#550: FILE: fpu/softfloat.c:7538:
+        if ( ( ( ( a.high | b.high ) << 1 ) | a.low | b.low ) == 0 ) {

ERROR: space prohibited after that open parenthesis '('
#559: FILE: fpu/softfloat.c:7579:
+        if ( aSig << 1 ) {

ERROR: space prohibited before that close parenthesis ')'
#559: FILE: fpu/softfloat.c:7579:
+        if ( aSig << 1 ) {

total: 142 errors, 3 warnings, 497 lines checked

Commit 16278b715204 (fpu/softfloat.c: add spaces around operator) has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20201114130715.1126922-1-zhangxinhao1@huawei.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com