* [OBV] amd64-features-makefile: add missing backslash.
@ 2017-09-20 15:09 Tedeschi, Walfred
[not found] ` <29be83e6-2984-487d-2f18-cd4df17c8c4c@redhat.com>
0 siblings, 1 reply; 3+ messages in thread
From: Tedeschi, Walfred @ 2017-09-20 15:09 UTC (permalink / raw)
To: gdb-patches
Hello All,
I have pushed this patch as obvious, basically there was no need to regenerate files.
Those files were modified by Yao already.
The change was only this:
-$(outdir)/i386/i386-avx-mpx-avx512-pku.dat: i386/32bit-core.xml
+$(outdir)/i386/i386-avx-mpx-avx512-pku.dat: i386/32bit-core.xml \
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=1b7fa39ed9678fded7264636c1e5cc14c6d6ebc6
Thanks and regards,
/Fred
Intel Deutschland GmbH
Registered Address: Am Campeon 10-12, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de
Managing Directors: Christin Eisenschmid, Christian Lamprechter
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [OBV] amd64-features-makefile: add missing backslash.
[not found] ` <29be83e6-2984-487d-2f18-cd4df17c8c4c@redhat.com>
@ 2017-09-20 16:00 ` Tedeschi, Walfred
2017-09-20 16:06 ` Pedro Alves
0 siblings, 1 reply; 3+ messages in thread
From: Tedeschi, Walfred @ 2017-09-20 16:00 UTC (permalink / raw)
To: Pedro Alves, gdb-patches
On 09/20/2017 05:56 PM, Pedro Alves wrote:
> On 09/20/2017 04:09 PM, Tedeschi, Walfred wrote:
>> Hello All,
>>
>> I have pushed this patch as obvious, basically there was no need to regenerate files.
>> Those files were modified by Yao already.
>>
>> The change was only this:
>> -$(outdir)/i386/i386-avx-mpx-avx512-pku.dat: i386/32bit-core.xml
>> +$(outdir)/i386/i386-avx-mpx-avx512-pku.dat: i386/32bit-core.xml \
>>
>> https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=1b7fa39ed9678fded7264636c1e5cc14c6d6ebc6
> We still need the fix (plus regeneration) on the 8.0 branch, right?
>
> Thanks,
> Pedro Alves
>
Yes, I have added the patch in the bugzilla. Can i commit also there?
Thanks and regards,
/Fred
Intel Deutschland GmbH
Registered Address: Am Campeon 10-12, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de
Managing Directors: Christin Eisenschmid, Christian Lamprechter
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [OBV] amd64-features-makefile: add missing backslash.
2017-09-20 16:00 ` Tedeschi, Walfred
@ 2017-09-20 16:06 ` Pedro Alves
0 siblings, 0 replies; 3+ messages in thread
From: Pedro Alves @ 2017-09-20 16:06 UTC (permalink / raw)
To: Tedeschi, Walfred, gdb-patches
On 09/20/2017 03:57 PM, Tedeschi, Walfred wrote:
> On 09/20/2017 05:56 PM, Pedro Alves wrote:
>> On 09/20/2017 04:09 PM, Tedeschi, Walfred wrote:
>>> Hello All,
>>>
>>> I have pushed this patch as obvious, basically there was no need to
>>> regenerate files.
>>> Those files were modified by Yao already.
>>>
>>> The change was only this:
>>> -$(outdir)/i386/i386-avx-mpx-avx512-pku.dat: i386/32bit-core.xml
>>> +$(outdir)/i386/i386-avx-mpx-avx512-pku.dat: i386/32bit-core.xml \
>>>
>>> https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=1b7fa39ed9678fded7264636c1e5cc14c6d6ebc6
>>>
>> We still need the fix (plus regeneration) on the 8.0 branch, right?
>
> Yes, I have added the patch in the bugzilla. Can i commit also there?
Yes, with proper commit log & ChangeLog (including a PR number reference).
Thanks,
Pedro Alves
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-09-20 16:06 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-20 15:09 [OBV] amd64-features-makefile: add missing backslash Tedeschi, Walfred
[not found] ` <29be83e6-2984-487d-2f18-cd4df17c8c4c@redhat.com>
2017-09-20 16:00 ` Tedeschi, Walfred
2017-09-20 16:06 ` Pedro Alves
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox