[PATCH ARC] Change default mach
Nick Clifton
nickc@redhat.com
Fri Feb 5 10:06:00 GMT 2016
Hi Cupertino Miranda,
> Looking forward for your review.
The patch itself is fine, but you missed one important thing: you did not
run regression tests to see if it introduces any new testsuite failures. If
you had you would have noticed that there was a new failure in the binutils
testsuite where it checks "objdump -f". The test file binutils/testsuite/
binutils-all/objdump.exp contains a list of expected architectures and your
patch changed the value for the ARC. Don't worry about it though - I have
included a fix in the patch I checked in.
> Cupertino Miranda <Cupertino.Miranda@synopsys.com>
One other extremely minor point: please could you put *two* spaces between
your name and your email address in changelog entries. IE:
Cupertino Miranda <Cupertino.Miranda@synopsys.com>
> * cpu-arc.c: Changed default archure from bfd_mach_arc_arcv2
> to bfd_mach_arc_arc600
Approved and applied.
Cheers
Nick
More information about the Binutils
mailing list