On Mon, Dec 12, 2022 at 8:42 PM Mark Wielaard wrote: > > Hi, > > On Fri, 2022-10-21 at 02:25 +0800, Yonggang Luo via Elfutils-devel > wrote: > > Signed-off-by: Yonggang Luo > > --- > > libcpu/i386_disasm.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libcpu/i386_disasm.c b/libcpu/i386_disasm.c > > index 599d1654..cc75a7b1 100644 > > --- a/libcpu/i386_disasm.c > > +++ b/libcpu/i386_disasm.c > > @@ -480,7 +480,7 @@ i386_disasm (Ebl *ebl __attribute__((unused)), > > > > /* gcc is not clever enough to see the following > > variables > > are not used uninitialized. */ > > - asm ("" > > + __asm ("" > > : "=mr" (opoff), "=mr" (correct_prefix), "=mr" > > (codep), > > "=mr" (next_curr), "=mr" (len)); > > } > > Urgh. Is this really (still) necessary? It is inside an if (0) block. > So it also is never used. Can we just get rid of the whole block? OK, I'll get rid of it > > Thanks, > > Mark -- 此致 礼 罗勇刚 Yours sincerely, Yonggang Luo