This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: Support Xilinx MicroBlaze processor - patch 1
- From: Michael Eager <eager at eagercon dot com>
- To: Nick Clifton <nickc at redhat dot com>
- Cc: "Joseph S. Myers" <joseph at codesourcery dot com>, binutils at sourceware dot org
- Date: Thu, 06 Aug 2009 12:26:57 -0700
- Subject: Re: Support Xilinx MicroBlaze processor - patch 1
- References: <4A7314FB.7040803@eagercon.com> <Pine.LNX.4.64.0907311733000.17822@digraph.polyomino.org.uk> <4A74DF49.3090906@eagercon.com> <4A7B15C1.6040002@redhat.com>
Nick Clifton wrote:
Hi Michael,
Cleaned comments, indents, braces.
Approved and applied.
Note - I have tidied up the formatting in various places.
A few things are missing from your patches:
* An entry in gas/NEWS mentioning the support for the new target.
Index: NEWS
===================================================================
RCS file: /cvs/src/src/gas/NEWS,v
retrieving revision 1.108
diff -r1.108 NEWS
22a23,24
> * Add support for Xilinx MicroBlaze architecture.
>
OK to check in ?
* Some microblaze specific tests in the gas testsuite, and possibly
the linker testsuite as well.
I'll see if I can come up with a couple tests.
I have also not applied the patch to config.sub - this must be submitted
separately to config-patches@gnu.org.
Will do.
I will apply the patch to the top level configure and configure.ac files
separately as this patch must be propagated into the FSF gcc source
repository as well.
Thanks. I'm cleaning up a patch to add MicroBlaze support to gcc,
but this is on a separate branch. There's no real problem with
applying the patch to configure and configure.ac in gcc head, but
it may give the incorrect impression that gcc head has support for
MicroBlaze.
--
Michael Eager eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306 650-325-8077