[PATCH v2] Work around the NOP issue of Loongson2F
Wu Zhangjin
wuzhangjin@gmail.com
Tue Nov 17 12:44:00 GMT 2009
On Tue, 2009-11-17 at 18:11 +0800, zhangfx wrote:
> >
> > Only one issue left, that is the nops instroduced by .align 5 or .fill
> > can not be replaced by our new nops. I have traced the source code, the
> > INSN_NOP is exactly replaced by "move at,at", but the stuff of .align
> > and .fill is a little complex, Seems they do not use the INSN_NOP, but
> > something else. I have tried to analyze the source code about
> > mips_align(), mips_handle_align() and mips_nop_opcode(), but I can not
> > easily get the result. any prompt here?
> >
> I think due to the source of the bug, we can leave alone .align, .fill
> implementations. Only when they are used in a sequence of short basic
> blocks they can be dangerous.
>
> e.g.
> beq L1;
> nop
> L1:
> beq L2;
> nop
> L2:
> beq L3;
> nop
> ...
>
Thanks, later, I will send the patch out without considering the .align
& .fill issue.
Regards,
Wu Zhangjin
More information about the Binutils
mailing list