[Lldb-commits] [PATCH] D67861: [LLDB] Check for the __MINGW32__ define instead of __MINGW64

2019-09-23 Thread Martin Storsjö via Phabricator via lldb-commits
mstorsjo added a comment. In D67861#1678618 , @labath wrote: > Judging by the microsoft docs > , > `vsnprintf` is

[Lldb-commits] [PATCH] D67861: [LLDB] Check for the __MINGW32__ define instead of __MINGW64

2019-09-23 Thread Pavel Labath via Phabricator via lldb-commits
labath added a reviewer: amccarth. labath added a comment. Judging by the microsoft docs , `vsnprintf` is avaliable at least since VS 2017 (the lowest version

[Lldb-commits] [PATCH] D67861: [LLDB] Check for the __MINGW32__ define instead of __MINGW64

2019-09-20 Thread Martin Storsjö via Phabricator via lldb-commits
mstorsjo created this revision. mstorsjo added reviewers: hhb, xen2, labath. Herald added subscribers: JDevlieghere, abidh. Herald added a project: LLDB. `__MINGW64__` is only defined for 64 bit targets, and the define without trailing underscores is never defined automatically (neither by clang