[PATCH ARC 3/3]: Fixed-linker-related-testsuite-for-ARC

Alan Modra amodra@gmail.com
Thu May 12 12:03:00 GMT 2016


On Thu, May 12, 2016 at 09:44:40AM +0000, Cupertino Miranda wrote:
> Hi Alan,
> 
> Thanks for noticing this mistake.
> Would it be reasonable to just change it to
> 
> +	 && ![istarget arc*-*-elf*]

Yes, I think so.  If I understand correctly from ld/emulparams
and ld/configure.tgt, arc-*-linux-uclibc* supports shared
libraries, but arc-*-elf* does not.

> Or should I change it in config.sub to default any elf* to elf.
> 
> The reason for the elf32 is our legacy build and regression testing
> scripts.
> Nevertheless, we do not consider it a requirement to keep this behavior
> specially if it could lead us to more of this problems in the future.
> 
> Regarding "./config.sub arc", should we make it default to something. We
> noticed that arm also gives "arm-unknown-none" and does not support the
> target.

ARM is quite different, supporting AOUT, COFF, PE and more binutils
formats.  ARC on the other hand, only has ELF support in binutils.
So it would be reasonable to change config.sub to translate plain
"arc" to "arc-elf".  I guess it's also quite reasonable to leave it
as is..  In any case, whether config.sub changes or not doesn't have
any bearing on this particular patch.

-- 
Alan Modra
Australia Development Lab, IBM



More information about the Binutils mailing list