Type: series Message-id: 20201008125659.49857-1-richard.henderson@linaro.org Subject: [PULL 00/11] tcg patch queue === 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/20200930121105.667049-1-eblake@redhat.com -> patchew/20200930121105.667049-1-eblake@redhat.com - [tag update] patchew/20201001170752.82063-1-richard.henderson@linaro.org -> patchew/20201001170752.82063-1-richard.henderson@linaro.org - [tag update] patchew/20201002215955.254866-1-richard.henderson@linaro.org -> patchew/20201002215955.254866-1-richard.henderson@linaro.org - [tag update] patchew/20201004180443.2035359-1-f4bug@amsat.org -> patchew/20201004180443.2035359-1-f4bug@amsat.org - [tag update] patchew/20201007160038.26953-1-alex.bennee@linaro.org -> patchew/20201007160038.26953-1-alex.bennee@linaro.org * [new tag] patchew/20201008125659.49857-1-richard.henderson@linaro.org -> patchew/20201008125659.49857-1-richard.henderson@linaro.org Switched to a new branch 'test' 9aed6a6 accel/tcg: Fix computing of is_write for MIPS 4d1e622 tcg: Remove TCG_TARGET_HAS_cmp_vec c4897cd tcg/optimize: Fold dup2_vec 2e58dc0 tcg: Fix generation of dupi_vec for 32-bit host 96f7941 tcg/i386: Fix dupi for avx2 32-bit hosts 0b5bcbb tcg: Remove TCGOpDef.used 40cea0e tcg: Move some TCG_CT_* bits to TCGArgConstraint bitfields 7c62277 tcg: Remove TCG_CT_REG 42d330b tcg: Move sorted_args into TCGArgConstraint.sort_index 1eff219 tcg: Drop union from TCGArgConstraint 2c84345 tcg: Adjust simd_desc size encoding === OUTPUT BEGIN === [34m[1m1/11[0m Checking commit 2c84345198c1 (tcg: Adjust simd_desc size encoding) [34m[1m2/11[0m Checking commit 1eff219a2edb (tcg: Drop union from TCGArgConstraint) [1m[31mERROR:[0m braces {} are necessary for all arms of this statement #407: FILE: tcg/tcg.c:2213: + if (tcg_regset_test_reg(arg_ct->regs, i)) [...] total: 1 errors, 0 warnings, 420 lines checked Patch 2/11 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. [34m[1m3/11[0m Checking commit 42d330b3f312 (tcg: Move sorted_args into TCGArgConstraint.sort_index) [34m[1m4/11[0m Checking commit 7c62277f9d05 (tcg: Remove TCG_CT_REG) [34m[1m5/11[0m Checking commit 40cea0e3c162 (tcg: Move some TCG_CT_* bits to TCGArgConstraint bitfields) [1m[31mERROR:[0m spaces prohibited around that ':' (ctx:WxW) #36: FILE: include/tcg/tcg.h:982: + unsigned ct : 16; ^ [1m[31mERROR:[0m spaces prohibited around that ':' (ctx:WxW) #37: FILE: include/tcg/tcg.h:983: + unsigned alias_index : 4; ^ [1m[31mERROR:[0m spaces prohibited around that ':' (ctx:WxW) #38: FILE: include/tcg/tcg.h:984: + unsigned sort_index : 4; ^ [1m[31mERROR:[0m spaces prohibited around that ':' (ctx:WxW) #39: FILE: include/tcg/tcg.h:985: + bool oalias : 1; ^ [1m[31mERROR:[0m spaces prohibited around that ':' (ctx:WxW) #40: FILE: include/tcg/tcg.h:986: + bool ialias : 1; ^ [1m[31mERROR:[0m spaces prohibited around that ':' (ctx:WxW) #41: FILE: include/tcg/tcg.h:987: + bool newreg : 1; ^ total: 6 errors, 0 warnings, 106 lines checked Patch 5/11 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. [34m[1m6/11[0m Checking commit 0b5bcbbf91ca (tcg: Remove TCGOpDef.used) [34m[1m7/11[0m Checking commit 96f79419c763 (tcg/i386: Fix dupi for avx2 32-bit hosts) [34m[1m8/11[0m Checking commit 2e58dc08335c (tcg: Fix generation of dupi_vec for 32-bit host) [34m[1m9/11[0m Checking commit c4897cd9e9b1 (tcg/optimize: Fold dup2_vec) [34m[1m10/11[0m Checking commit 4d1e62279de6 (tcg: Remove TCG_TARGET_HAS_cmp_vec) [34m[1m11/11[0m Checking commit 9aed6a69f26e (accel/tcg: Fix computing of is_write for MIPS) [1m[35mWARNING:[0m architecture specific defines should be avoided #28: FILE: accel/tcg/user-exec.c:705: +#if defined(__misp16) || defined(__mips_micromips) [1m[31mERROR:[0m space required before the open parenthesis '(' #44: FILE: accel/tcg/user-exec.c:719: + switch((insn >> 26) & 077) { [1m[35mWARNING:[0m architecture specific defines should be avoided #57: FILE: accel/tcg/user-exec.c:732: +#if !defined(__mips_isa_rev) || __mips_isa_rev < 6 [1m[35mWARNING:[0m Block comments use a leading /* on a separate line #64: FILE: accel/tcg/user-exec.c:739: + /* Required in all versions of MIPS64 since [1m[35mWARNING:[0m Block comments use * on subsequent lines #65: FILE: accel/tcg/user-exec.c:740: + /* Required in all versions of MIPS64 since + MIPS64r1 and subsequent versions of MIPS32r2. */ [1m[35mWARNING:[0m Block comments use a trailing */ on a separate line #65: FILE: accel/tcg/user-exec.c:740: + MIPS64r1 and subsequent versions of MIPS32r2. */ total: 1 errors, 5 warnings, 55 lines checked Patch 11/11 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