This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Hi Marcus, 1) I have attached the binutils patch in the first mail (https://sourceware.org/ml/binutils/2016-06/msg00064.html) Attaching in this mail too . Corresponding GCC patch is here https://gcc.gnu.org/ml/gcc-patches/2016-06/msg00541.html 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 with regards, Virendra Pathak On Wed, Jun 8, 2016 at 4:58 AM, Marcus Shawcroft <marcus.shawcroft@gmail.com> wrote: > On 7 June 2016 at 14:10, Syed Faisal Akber <faisal@akber.net> wrote: >> >> >> Sent from my BlackBerry 10 smartphone. >> Original Message >> From: Virendra Pathak >> Sent: Tuesday, June 7, 2016 04:56 >> To: binutils@sourceware.org >> Cc: Virendra Kumar Pathak; nickc@redhat.com >> Subject: add vulcan support to gas >> >> Hi binutils group, >> >> We have a patch for adding -mcpu=vulcan support to gas. >> Broadcom's Vulcan is a armv8-a aarch64 served based processor. >> We have also got company wide FSF agreement for contributing to >> binutils project. >> >> However, recently I read on this group that now gcc does not pass >> -mcpu flag to binutils. GCC now passes -march flag to binutils. >> I verified this by compiling a simple hello world inspecting the >> assembler flags. >> [https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01106.html] >> [https://sourceware.org/ml/binutils/2015-08/msg00169.html] >> >> Keeping this in mind, do we still need to add -mcpu=vulcan support to >> gas, given that many armv8-a vendors have done it? >> Kindly guide me on this. >> >> Please also find the patch (-mcpu=vulcan) attached with this mail for review. > > Hi, The patch appears to be missing ? > > >> Tested the patch with aarch64-linux-gnu cross build, >> aarch64-unknown-linux-gnu native build and make check (ld,gdb,gas,binutils). >> Also verified the new option via command line. >> >> Thanks for your time. >> >> >> gas/ChangeLog: >> * config/tc-aarch64.c (aarch64_cpus): Add vulcan. >> * doc/c-aarch64.texi : Document that vulcan is a valid processor name > > The proposed changelog entry is missing the author line. > > Thanks > /Marcus > >> >> >> with regards, >> Virendra Pathak
Attachment:
0001-aarch64-add-vulcan-support-to-gas.patch.txt
Description: Text document
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |