This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
On 26/07/13 03:10, John Tytgat wrote:
--- ld/testsuite/ld-arm/group-relocs-ldrs-bad.s 15 Jun 2006 11:03:02 -0000 1.1 +++ ld/testsuite/ld-arm/group-relocs-ldrs-bad.s 14 Jul 2013 16:45:03 -0000 @@ -6,8 +6,8 @@ .globl _start _start: - add r0, r0, #:sb_g0_nc:(bar) - ldrd r2, [r0, #:sb_g1:(bar)] + add r0, r0, #:pc_g0_nc:(bar) + ldrd r2, [r0, #:pc_g1:(bar + 4)] @ We will place the section foo at 0x8000100.Why have you changed this test from SB relative to PC relative? That means it no-longer tests the same thing.We originally had 4 linker groups tests : group-relocs-alu-bad, group-relocs-ldc-bad, group-relocs-ldr-bad and group-relocs-ldrs-bad. The first two were pc-based, the last two were sb-based. I've changed the last two in pc-based as well and added 4 new sb-based tests. John.
I think the 's' in the test name (ldrs) is supposed to indicate that this is sb based. I'd rather you add new tests than changed existing ones. It's not as though the gas testsuite takes a long time to run (relative to the compiler at any rate).
R.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |