This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH x86_64][BZ #20139] Don't allow configure with not supporting AVX512 assembler w/o --disable-avx512.
- From: Andrew Senkevich <andrew dot n dot senkevich at gmail dot com>
- To: Florian Weimer <fweimer at redhat dot com>
- Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, Joseph Myers <joseph at codesourcery dot com>, libc-alpha <libc-alpha at sourceware dot org>
- Date: Fri, 1 Jul 2016 15:28:41 +0300
- Subject: Re: [PATCH x86_64][BZ #20139] Don't allow configure with not supporting AVX512 assembler w/o --disable-avx512.
- Authentication-results: sourceware.org; auth=none
- References: <CAMXFM3tJkhW78PPe4=FUhbm=8HoOY4iouSqXxLObA0GEMTxoZA at mail dot gmail dot com> <alpine dot DEB dot 2 dot 20 dot 1606271751470 dot 7438 at digraph dot polyomino dot org dot uk> <CAMXFM3t_92RMmU=u-70nCHqwXL0BsZ6=82Jn=neiDx82HxjjRg at mail dot gmail dot com> <96d24e9d-bf84-a43b-2860-c49d4fe341a4 at redhat dot com> <CAMe9rOqrRTTs=oMHoew3tUtS+ru2WvWfHuOyidOOMaV468SEZw at mail dot gmail dot com> <alpine dot DEB dot 2 dot 20 dot 1606272019240 dot 3978 at digraph dot polyomino dot org dot uk> <CAMXFM3v586M2XBqf53KHGsfSLV67tuJ1nrSBez5JQ--x9ZtWpQ at mail dot gmail dot com> <CAMe9rOr7BgZjgEH5NwY-utDZSjDWpu1wzMCATMjw0WrvK_NP9Q at mail dot gmail dot com> <c198f016-9b0d-f3e5-21a1-b4e54b210b35 at redhat dot com> <CAMe9rOp3m68KQPFkDCu+ckETGGeQjEot=b7v0zY9g-bPPVk6eg at mail dot gmail dot com> <53986bbf-b12c-7d68-01de-5506ddf6d6e0 at redhat dot com>
2016-07-01 9:13 GMT+03:00 Florian Weimer <fweimer@redhat.com>:
> On 06/30/2016 02:23 PM, H.J. Lu wrote:
>>
>> On Thu, Jun 30, 2016 at 4:15 AM, Florian Weimer <fweimer@redhat.com>
>> wrote:
>>>
>>> On 06/29/2016 11:45 PM, H.J. Lu wrote:
>>>>
>>>>
>>>> Subject: [PATCH] Require binutils 2.24 to build x86-64 glibc
>>>
>>>
>>>
>>> Please put [BZ #20139] at the end of the first line.
>>
>>
>> What do you mean by that? My ChangeLog entries are
>>
>> [BZ #20139]
>> * config.h.in (HAVE_AVX512_ASM_SUPPORT): Renamed to ...
>> (HAVE_AVX512DQ_ASM_SUPPORT): This.
>>
>> It looks normal to me.
>
>
> I meant at the end of the first line of the commit message.
>
>> Here is the patch with updated commit message, including only disabling
>> the AVX512DQ part of mathvec without AVX512DQ.
>>
>> OK for master?
>
>
> I have no objections to the patch as such (but also see my other comment
> about aligning the XGETBV values with what the dynamic linker supports).
Have we decided how to fix it in 2.23 release (which contains this bug)?
--
WBR,
Andrew