[Lldb-commits] [PATCH] D67894: [LLDB] Rework a MinGW build fix from D65691

2019-09-23 Thread Martin Storsjö via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL372656: [LLDB] Rework a MinGW build fix from D65691 (authored by mstorsjo, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

[Lldb-commits] [PATCH] D67894: [LLDB] Rework a MinGW build fix from D65691

2019-09-23 Thread Haibo Huang via Phabricator via lldb-commits
hhb accepted this revision. hhb added a comment. This revision is now accepted and ready to land. Thanks! Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67894/new/ https://reviews.llvm.org/D67894 ___ lldb-commits

[Lldb-commits] [PATCH] D67894: [LLDB] Rework a MinGW build fix from D65691

2019-09-23 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. Let's wait for @hhb to give this a spin to avoid twiddling this back and forth. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67894/new/ https://reviews.llvm.org/D67894 ___ lldb-commits

[Lldb-commits] [PATCH] D67894: [LLDB] Rework a MinGW build fix from D65691

2019-09-22 Thread Martin Storsjö via Phabricator via lldb-commits
mstorsjo created this revision. mstorsjo added a reviewer: hhb. Herald added subscribers: JDevlieghere, abidh. Herald added a project: LLDB. That change didn't contain any explanation for this bit. There shouldn't be any need for a check for MinGW ifdefs here, as long as the include uses