This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH][MIPS] Cleanup expected output for several MIPS targets
- From: Richard Sandiford <rdsandiford at googlemail dot com>
- To: Matthew Fortune <Matthew dot Fortune at imgtec dot com>
- Cc: "Maciej W. Rozycki" <macro at codesourcery dot com>, "binutils\ at sourceware dot org" <binutils at sourceware dot org>, "Eric Christopher \(echristo\ at gmail dot com\)" <echristo at gmail dot com>, Alan Modra <amodra at gmail dot com>, Takashi Yoshii <yoshii dot takashi at renesas dot com>
- Date: Wed, 03 Sep 2014 19:43:06 +0100
- Subject: Re: [PATCH][MIPS] Cleanup expected output for several MIPS targets
- Authentication-results: sourceware.org; auth=none
- References: <6D39441BF12EF246A7ABCE6654B0235320F0301C at LEMAIL01 dot le dot imgtec dot org>
Matthew Fortune <Matthew.Fortune@imgtec.com> writes:
> Attached is a testsuite-only patch to ensure all the tests added in the
> MIPS O32 FPXX patch will pass on all MIPS targets.
>
> This probably amounts to an obvious fix even though it is quite large
> but I would like to double check the changes I made to the mips.exp and
> mips-elf.exp files are OK.
Not sure about the obviousness :-) There seem to be several different
types of change here. Please could you split the patch up so that there's
one type of change per patch? E.g. adding extra sections is simple enough,
but it's less obvious that:
diff --git a/gas/testsuite/gas/mips/attr-gnu-abi-fp-1.d b/gas/testsuite/gas/mips/attr-gnu-abi-fp-1.d
index ce5bbc2..17218d3 100644
--- a/gas/testsuite/gas/mips/attr-gnu-abi-fp-1.d
+++ b/gas/testsuite/gas/mips/attr-gnu-abi-fp-1.d
@@ -9,14 +9,14 @@ File Attributes
MIPS ABI Flags Version: 0
-ISA: MIPS1
+ISA: MIPS.*
GPR size: 32
CPR1 size: 32
CPR2 size: 0
FP ABI: Hard float \(double precision\)
-ISA Extension: None
+ISA Extension: .*
ASEs:
None
-FLAGS 1: 00000000
+FLAGS 1: 0000000.
FLAGS 2: 00000000
is a good idea, since in some ways it weakens the test. Another option
would be to force MIPS I on the command line.
The mips.exp change is OK as a stand-alone patch.
Why is:
@@ -60,7 +60,7 @@ set linux_gnu [expr [istarget mips*-*-linux*]]
set embedded_elf [expr [istarget mips*-*-elf]]
# Set defaults.
-set abi_asflags(o32) ""
+set abi_asflags(o32) "-32"
set abi_asflags(n32) "-march=from-abi -n32 -EB"
set abi_asflags(n64) "-march=from-abi -64 -EB"
set abi_ldflags(o32) ""
needed now and not before?
Thanks,
Richard