Re: add vulcan support to gas

[Virendra Pathak <virendra dot pathak at broadcom dot com>]

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