[Lldb-commits] [PATCH] D67910: [LLDB] Avoid warnings about redefining posix mode defines on MinGW
Martin Storsjö via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Sep 23 23:23:18 PDT 2019
mstorsjo marked an inline comment as done.
mstorsjo added inline comments.
Comment at: lldb/include/lldb/Host/windows/PosixApi.h:39
#define S_IRGRP 0 /* read, group */
> Maybe move these into the defined(_MSC_VER) above? Or the other way? Seems they should be consistent.
I don't want to move them into the `_MSC_VER` ifdef above, as there are pretty recent mingw-w64 releases without these new defines as well (the new ones were only added in mingw-w64 v4), but I guess I could change the `_MSC_VER` ifdef to check for `S_IRUSR` instead.
CHANGES SINCE LAST ACTION
More information about the lldb-commits