[PATCH 00/12] Xilinx MicroBlaze Support Update

[Joseph S. Myers]

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