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:
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
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
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