This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 00/12] Xilinx MicroBlaze Support Update
On Wed, 10 Oct 2012, Joseph S. Myers wrote:
> On Wed, 10 Oct 2012, david.holsgrove@xilinx.com wrote:
>
> > This patch series updates binutils support for Xilinx MicroBlaze by
> > adding support for new instructions, little endian target (microblazeel)
> > and TLS relocations.
>
> These patches all seem to be missing testcases for the new or changed or
> bug-fixed features, which should be added to the assembler and linker
> testsuites.
... and I see that actually the assembler and linker testsuites are
completely missing any tests for MicroBlaze, indicating an unfortunate
failure of patch review when the port went in originally. So the lack of
testcases probably isn't a reason to prevent these patches going in - but
I strongly advise adding reasonable testsuite coverage (testing each
instruction / relocation in the assembler, each relocation in the linker,
each architecture-specific error message, etc.), as otherwise it's quite
likely MicroBlaze could accidentally get broken without this readily being
noticed.
--
Joseph S. Myers
joseph@codesourcery.com