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 bfd]: Add further support for x86_64 mingw


Kai Tietz wrote:

You are right, that the simple chack for __MSVCRT__ isn't engough, but the pattern
#if defined (_WIN32) && !defined (__CYGWIN__)
seems to be a bit misleading.

grep through the binutils sources for _WIN32 or "_WIN32.*__CYGWIN" ? It's what's normally used.

> IMHO the clause should be something like
#if defined(__CRTDLL__) || defined(__MSVCRT__)

This pattern matches for crtdll (ce), uwin, cygwin, and mingw.
Do you agree?


Well, it doesn't match ce (coredll not crtdll there), but then again, there's no such thing as a native binutils CE port. I was only pointing it out as example of the nature of that macro. I've never really used crtdll -- it's all msvcrt nowadays, AFAIK. Anyway, this tends to bikeshed, so, I'll just state that whatever goes in is fine by me.

--
Pedro Alves


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