[patch] Add MIPS nonpic tests to testsuite
Fri Jan 30 18:22:00 GMT 2009
I finally got around to testing out this suggestion. The mips nonpic tests pass with this patch.
Okay to install?
2009-01-30 Catherine Moore <email@example.com>
* sde.h (SUBTARGET_ARM_SPEC): Don;t assemble -fpic code as
--- sde.h (revision 143798)
+++ sde.h (working copy)
@@ -60,7 +60,6 @@ along with GCC; see the file COPYING3.
#define SUBTARGET_ASM_SPEC "\
Richard Sandiford wrote:
> Catherine Moore <firstname.lastname@example.org> writes:
>> Richard Sandiford wrote:
>>> Adam Nemet <email@example.com> writes:
>>>> Catherine Moore <firstname.lastname@example.org> writes:
>>>>> I've tested with mips-linux and also with mips-sde-elf. The testsuite
>>>>> doesn't currently pass with the mips-sde-elf configuration due to extra
>>>>> compiler/linker messages, though.
>>>> Can you elaborate? It would be nice if the new tests passed on the
>>>> configurations that they're run on.
>>> I imagine it was:
>>> in sde.h:SUBTARGET_ASM_SPEC. So we assemble -fpic code as -mabicalls,
>>> even though GCC hasn't generated that.
>>> If so, it looks like an SDE bug to me, and it's good that the testsuite
>>> has picked up on it. -fpic really ought to be link-compatible with
>>> -fno-pic, if it's accepted at all.
>> Actually, the problem that I'm currently seeing is:
>> /tmp/cc6v0Red.s: Assembler messages:^M
>> /tmp/cc6v0Red.s:26: Warning: No .cprestore pseudo-op used in PIC code^M
>> /tmp/cc6v0Red.s:26: Warning: Macro instruction expanded into multiple instructions^M
> Yeah, this would be a symptom of the problem I mentioned.
> The assembler thinks that it's compiling abicalls code,
> but GCC has compiled non-abicalls code.
> IMO, we should just remove that line from sde.h:SUBTARGET_ASM_SPEC.
More information about the Gcc-patches