When a function-like macro expands to an expression, that expression doesn't need a semicolon after it. All uses have been verified to have their own semicolons. This was found using the following Coccinelle semantic patch: @r@ identifier i : script:ocaml() { String.lowercase_ascii i = i }; expression e; @@ *#define i(...) e; Signed-off-by: Julia Lawall --- arch/sparc/net/bpf_jit_comp_32.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/sparc/net/bpf_jit_comp_32.c b/arch/sparc/net/bpf_jit_comp_32.c index bda2dbd3f..632002681 100644 --- a/arch/sparc/net/bpf_jit_comp_32.c +++ b/arch/sparc/net/bpf_jit_comp_32.c @@ -262,13 +262,13 @@ do { *prog++ = BR_OPC | WDISP22(OFF); \ *prog++ = (SUBCC | RS1(R1) | RS2(R2) | RD(G0)) #define emit_cmpi(R1, IMM) \ - *prog++ = (SUBCC | IMMED | RS1(R1) | S13(IMM) | RD(G0)); + *prog++ = (SUBCC | IMMED | RS1(R1) | S13(IMM) | RD(G0)) #define emit_btst(R1, R2) \ *prog++ = (ANDCC | RS1(R1) | RS2(R2) | RD(G0)) #define emit_btsti(R1, IMM) \ - *prog++ = (ANDCC | IMMED | RS1(R1) | S13(IMM) | RD(G0)); + *prog++ = (ANDCC | IMMED | RS1(R1) | S13(IMM) | RD(G0)) #define emit_sub(R1, R2, R3) \ *prog++ = (SUB | RS1(R1) | RS2(R2) | RD(R3))