This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] microMIPS: Branch swapping support
- From: Richard Sandiford <rdsandiford at googlemail dot com>
- To: "Maciej W. Rozycki" <macro at codesourcery dot com>
- Cc: binutils at sourceware dot org, Chao-ying Fu <fu at mips dot com>, Rich Fuhler <rich at mips dot com>, David Lau <davidlau at mips dot com>, Kevin Mills <kevinm at mips dot com>, Ilie Garbacea <ilie at mips dot com>, Catherine Moore <clm at codesourcery dot com>, Nathan Sidwell <nathan at codesourcery dot com>, Joseph Myers <joseph at codesourcery dot com>
- Date: Sat, 06 Aug 2011 11:37:46 +0100
- Subject: Re: [PATCH] microMIPS: Branch swapping support
- References: <alpine.DEB.1.10.1108021835260.4083@tp.orcam.me.uk>
"Maciej W. Rozycki" <macro@codesourcery.com> writes:
> /* We have no flag to mark the read from "y", so we use TRAP to disable
> delay slot scheduling of ALNV.PS altogether. */
> {"alnv.ps", "D,V,T,y", 0x54000019, 0xfc00003f, TRAP|WR_D|RD_S|RD_T|FP_D, 0, I1 },
> -{"and", "mf,mt,mg", 0x4480, 0xffc0, 0, MOD_mf|MOD_mg, I1 },
> -{"and", "mf,mg,mx", 0x4480, 0xffc0, 0, MOD_mf|MOD_mg, I1 },
> +{"and", "mf,mt,mg", 0x4480, 0xffc0, 0, WR_mf|RD_mg, I1 },
> +{"and", "mf,mg,mx", 0x4480, 0xffc0, 0, WR_mf|RD_mg, I1 },
Unless I'm missing something, it looks like this loses the read from mf.
Same for OR and XOR.
Richard