[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 0/4] softfloat patch queue
From: |
no-reply |
Subject: |
Re: [PULL 0/4] softfloat patch queue |
Date: |
Fri, 15 May 2020 22:46:14 -0700 (PDT) |
Patchew URL: https://patchew.org/QEMU/address@hidden/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Message-id: address@hidden
Subject: [PULL 0/4] softfloat patch queue
Type: series
=== 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/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
36efd04 softfloat: fix floatx80 pseudo-denormal round to integer
4c2e880 softfloat: fix floatx80 pseudo-denormal comparisons
013fb4d softfloat: fix floatx80 pseudo-denormal addition / subtraction
f1a96f5 softfloat: silence sNaN for conversions to/from floatx80
=== OUTPUT BEGIN ===
1/4 Checking commit f1a96f5122b8 (softfloat: silence sNaN for conversions
to/from floatx80)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#88:
new file mode 100644
ERROR: Use of volatile is usually wrong, please add a comment
#98: FILE: tests/tcg/i386/test-i386-snan-convert.c:6:
+volatile float f_res;
ERROR: Use of volatile is usually wrong, please add a comment
#99: FILE: tests/tcg/i386/test-i386-snan-convert.c:7:
+volatile double d_res;
ERROR: Use of volatile is usually wrong, please add a comment
#100: FILE: tests/tcg/i386/test-i386-snan-convert.c:8:
+volatile long double ld_res;
ERROR: Use of volatile is usually wrong, please add a comment
#102: FILE: tests/tcg/i386/test-i386-snan-convert.c:10:
+volatile float f_snan = __builtin_nansf("");
ERROR: Use of volatile is usually wrong, please add a comment
#103: FILE: tests/tcg/i386/test-i386-snan-convert.c:11:
+volatile double d_snan = __builtin_nans("");
ERROR: Use of volatile is usually wrong, please add a comment
#104: FILE: tests/tcg/i386/test-i386-snan-convert.c:12:
+volatile long double ld_snan = __builtin_nansl("");
total: 6 errors, 1 warnings, 123 lines checked
Patch 1/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
2/4 Checking commit 013fb4df5f1b (softfloat: fix floatx80 pseudo-denormal
addition / subtraction)
WARNING: Block comments use a leading /* on a separate line
#29: FILE: fpu/softfloat.c:5870:
+ /* At least one of the values is a pseudo-denormal,
WARNING: Block comments use a trailing */ on a separate line
#30: FILE: fpu/softfloat.c:5871:
+ * and there is a carry out of the result. */
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#38:
new file mode 100644
ERROR: Use of volatile is usually wrong, please add a comment
#53: FILE: tests/tcg/i386/test-i386-pseudo-denormal.c:11:
+volatile union u ld_pseudo_m16382 = { .s = { UINT64_C(1) << 63, 0 } };
ERROR: Use of volatile is usually wrong, please add a comment
#55: FILE: tests/tcg/i386/test-i386-pseudo-denormal.c:13:
+volatile long double ld_res;
ERROR: spaces required around that '-' (ctx:VxV)
#61: FILE: tests/tcg/i386/test-i386-pseudo-denormal.c:19:
+ if (ld_res != 0x1p-16381L) {
^
total: 3 errors, 3 warnings, 36 lines checked
Patch 2/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
3/4 Checking commit 4c2e8807fd1f (softfloat: fix floatx80 pseudo-denormal
comparisons)
ERROR: spaces required around that '-' (ctx:VxV)
#43: FILE: tests/tcg/i386/test-i386-pseudo-denormal.c:23:
+ if (ld_pseudo_m16382.ld != 0x1p-16382L) {
^
total: 1 errors, 0 warnings, 22 lines checked
Patch 3/4 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/4 Checking commit 36efd04e9c7b (softfloat: fix floatx80 pseudo-denormal round
to integer)
ERROR: space prohibited after that open parenthesis '('
#29: FILE: fpu/softfloat.c:5744:
+ && ( (uint64_t) ( extractFloatx80Frac( a ) ) == 0 ) ) {
ERROR: space prohibited before that close parenthesis ')'
#29: FILE: fpu/softfloat.c:5744:
+ && ( (uint64_t) ( extractFloatx80Frac( a ) ) == 0 ) ) {
total: 2 errors, 0 warnings, 29 lines checked
Patch 4/4 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/address@hidden/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to address@hidden
- [PULL 0/4] softfloat patch queue, Richard Henderson, 2020/05/15
- [PULL 1/4] softfloat: silence sNaN for conversions to/from floatx80, Richard Henderson, 2020/05/15
- [PULL 2/4] softfloat: fix floatx80 pseudo-denormal addition / subtraction, Richard Henderson, 2020/05/15
- [PULL 4/4] softfloat: fix floatx80 pseudo-denormal round to integer, Richard Henderson, 2020/05/15
- [PULL 3/4] softfloat: fix floatx80 pseudo-denormal comparisons, Richard Henderson, 2020/05/15
- Re: [PULL 0/4] softfloat patch queue, Peter Maydell, 2020/05/15
- Re: [PULL 0/4] softfloat patch queue,
no-reply <=