[PATCH] D62533: Build with _XOPEN_SOURCE defined on AIX

2019-06-07 Thread David Tenty via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL362808: Build with _XOPEN_SOURCE defined on AIX (authored by daltenty, committed by ). Changed prior to commit: https://reviews.llvm.org/D62533?vs=202822&id=203563#toc Repository: rL LLVM CHANGES SI

[PATCH] D62533: Build with _XOPEN_SOURCE defined on AIX

2019-06-03 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast accepted this revision. hubert.reinterpretcast added a comment. This revision is now accepted and ready to land. @daltenty, I believe that you have had a good number of patches committed into the project already (rL360898 , rL361410

[PATCH] D62533: Build with _XOPEN_SOURCE defined on AIX

2019-06-03 Thread David Tenty via Phabricator via cfe-commits
daltenty updated this revision to Diff 202822. daltenty added a comment. Fix grammar in comments. Fix up the way we define the macros for Google Test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62533/new/ https://reviews.llvm.org/D62533 Files:

[PATCH] D62533: Build with _XOPEN_SOURCE defined on AIX

2019-05-28 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast added inline comments. Comment at: llvm/CMakeLists.txt:862 +# Build with _XOPEN_SOURCE on AIX, as stray macros in _ALL_SOURCE mode tend to +# break things. In this case we need to enable the LARGE FILE API as well +if (UNIX AND ${CMAKE_SYSTEM_NAME} MATCHES

[PATCH] D62533: Build with _XOPEN_SOURCE defined on AIX

2019-05-28 Thread David Tenty via Phabricator via cfe-commits
daltenty created this revision. daltenty added reviewers: hubert.reinterpretcast, xingxue, andusy. daltenty added projects: LLVM, clang. Herald added subscribers: llvm-commits, cfe-commits, jsji, mgorny. It is useful to build with _XOPEN_SOURCE defined on AIX, enabling X/Open and POSIX compatibili