Re: add vulcan support to gas

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

Please find the ChangeLog in the correct format.

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


Thanks.

with regards,
Virendra Pathak


On Wed, Jun 8, 2016 at 11:05 AM, Virendra Pathak
<virendra.pathak@broadcom.com> wrote:
> 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