[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