This is the mail archive of the
mailing list for the binutils project.
Re: [PATCH v2] Work around the NOP issue of Loongson2F
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.
> beq L1;
> beq L2;
> beq L3;
Thanks, later, I will send the patch out without considering the .align
& .fill issue.