fpu/softfloat.c | 150 ++++++++++++++++++++++++------------------------ 1 file changed, 75 insertions(+), 75 deletions(-)
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
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
© 2016 - 2026 Red Hat, Inc.