add vulcan support to gas
Marcus Shawcroft
marcus.shawcroft@gmail.com
Wed Jun 8 08:00:00 GMT 2016
On 8 June 2016 at 06:35, Virendra Pathak <virendra.pathak@broadcom.com> wrote:
> Hi Marcus,
>
> 1) I have attached the binutils patch in the first mail
Hi,
Thanks for the patch, I have a couple of minor comments:
+ {"vulcan", AARCH64_FEATURE (AARCH64_ARCH_V8,
+ AARCH64_FEATURE_CRC | AARCH64_FEATURE_CRYPTO),
+ "Broadcom Vulcan"},
{"thunderx", AARCH64_FEATURE (AARCH64_ARCH_V8,
Please add vulcan in alphabetical order.
@code{qdf24xx},
+@code{vulcan},
@code{thunderx},
Likewise.
> 2) Added the author entry in the ChangeLog
>
> gas/ChangeLog:
> Virendra Pathak (virendra.pathak@broadcom.com)
> * config/tc-aarch64.c (aarch64_cpus): Add vulcan.
> * doc/c-aarch64.texi : Document that vulcan is a valid processor name
No space between before the ':', full stop at the end of the sentence.
Thanks
/Marcus
More information about the Binutils
mailing list