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]

Re: [PATCH] [PowerPC VLE] Add LSP instructions support


>> If you can commit with that fixed, please go ahead.  If
>> not, no need to post another patch.  I'll commit it for you with the
>> ChangeLog fixed.

Yes please, I have no permission to push on binutils-gdb.git

Best regards,
Alex

On Mon, Aug 21, 2017 at 3:45 PM, Alexander Fedotov <alfedotov@gmail.com> wrote:
>> This is still incorrect.  Paths are supposed to be relative to the
>> directory containing the ChangeLog file, so you need to lose the
>> gas/ prefix.  If you can commit with that fixed, please go ahead.  If
>> not, no need to post another patch.  I'll commit it for you with the
>> ChangeLog fixed.
>
> Well I have no write permissions.
> Fixed :)
>
>
>
> On Mon, Aug 21, 2017 at 3:31 PM, Alan Modra <amodra@gmail.com> wrote:
>> On Mon, Aug 21, 2017 at 01:12:37PM +0300, Alexander Fedotov wrote:
>>> Hello Alan
>>>
>>> Please find attached new version.
>>>
>>> BTW I found some mess with tabs and spaces in ppc-opc.c
>>
>> Yes, I see.  Indentation is supposed to be tabs+spaces.  (You can
>> infer that from the GNU coding standard saying that it recommends the
>> default style of the "indent" program.  However, as far as I know the
>> coding standard doesn't call for tabs explicitly, so there is some
>> room for dissenting opinion.)
>>
>>> --- a/gas/ChangeLog
>>> +++ b/gas/ChangeLog
>>> @@ -1,3 +1,13 @@
>>> +2017-08-19  Alexander Fedotov <alexander.fedotov@nxp.com>
>>> +         Edmar Wienskoski <edmar.wienskoski@nxp.com
>>> +
>>> +     * gas/testsuite/gas/ppc/lsp-checks.d: New test
>>> +     * gas/testsuite/gas/ppc/lsp-checks.l: New
>>> +     * gas/testsuite/gas/ppc/lsp-checks.s: New
>>> +     * gas/testsuite/gas/ppc/lsp.d: New
>>> +     * gas/testsuite/gas/ppc/lsp.s: New
>>> +     * gas/testsuite/gas/ppc/ppc.exp: Run new tests.
>>
>> This is still incorrect.  Paths are supposed to be relative to the
>> directory containing the ChangeLog file, so you need to lose the
>> gas/ prefix.  If you can commit with that fixed, please go ahead.  If
>> not, no need to post another patch.  I'll commit it for you with the
>> ChangeLog fixed.
>>
>> --
>> Alan Modra
>> Australia Development Lab, IBM
>
>
>
> --
> Best regards,
> AF



-- 
Best regards,
AF


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]