[llvm] r303272 - Workaround for incorrect Win32 header on GCC.

İsmail Dönmez via llvm-commits llvm-commits at lists.llvm.org
Thu May 18 10:00:57 PDT 2017


Luckily for us, this seems to be reported & fixed yesterday :)
https://sourceforge.net/p/mingw-w64/bugs/614/

On Thu, May 18, 2017 at 6:40 PM, Zachary Turner <zturner at google.com> wrote:
> Considering it's such a simple patch, it would help me out quite a bit if
> you could do it.  I don't even have a MinGW checkout or installation on my
> machine, so to create a patch I'd have to go through a lot of hoops for what
> is effectively a one-character change.
>
> Thanks!
>
> On Thu, May 18, 2017 at 4:40 AM İsmail Dönmez <ismail at i10z.com> wrote:
>>
>> Hi,
>>
>> On Wed, May 17, 2017 at 8:09 PM, Zachary Turner <zturner at google.com>
>> wrote:
>> > To be honest I haven't checked.  The incorrect definition is here:
>> >
>> >
>> > https://sourceforge.net/p/mingw-w64/mingw-w64/ci/d43f63b334eac3eaf6ed66f2206fee5eb67064ec/tree/mingw-w64-headers/include/ntstatus.h#l164
>> >
>> > You can see that there are unbalanced parentheses in the #define.
>> >
>> > I'm not familiar with the MinGW / GCC community, but if it's
>> > straightforward
>> > and you point me to the right place and I can submit a bug report.
>>
>> Thanks for pointing out the line. This should be easy to get fixed
>> upstream. Just send a patch to mingw-w64-public at lists.sourceforge.net
>> mailing list (probably members only). If you can't do that I'll do
>> that and reference you in the patch.
>>
>> Regards,
>> ismail


More information about the llvm-commits mailing list