Ping version 2 of the MIPS soft-float patch
Eric Christopher
echristo@apple.com
Mon Apr 28 18:06:00 GMT 2008
On Apr 28, 2008, at 8:21 AM, Adam Nemet wrote:
> http://sourceware.org/ml/binutils/2008-04/msg00349.html
I'd been waiting for Richard to comment since he had last time, but...
Couple of comments:
Can you comment this struct with the field locations please?
static struct mips_set_options mips_opts = {
- ISA_UNKNOWN, -1, -1, 0, -1, -1, -1, -1, 0, ATREG, 0, 0, 0, 0, 0, 0,
CPU_UNKNOWN, FALSE
+ ISA_UNKNOWN, -1, -1, 0, -1, -1, -1, -1, 0, ATREG, 0, 0, 0, 0, 0, 0,
+ CPU_UNKNOWN, FALSE, FALSE, FALSE };
and the general flow here, i.e. what's going on. It's fairly obvious
but a comment would be nice.
+ if (mo->pinfo == INSN_MACRO)
Otherwise OK.
-eric
More information about the Binutils
mailing list