[PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

shafik added a comment. Herald added a subscriber: steven.zhang. Herald added a project: All. @hubert.reinterpretcast is this still relevant? It looks like portions of this have landed, can we close it? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D18360/new/ https://reviews.llvm.org/D

[PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

hubert.reinterpretcast added a comment. D59048 has been posted with some initial `OSTargetInfo` changes. Noticeable differences include not defining `_ALL_SOURCE` and focus on 64-bit long double. This is consistent with the invocation provided with the recent v

[PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

hubert.reinterpretcast added a comment. In D18360#1406455 , @apaprocki wrote: > @hubert.reinterpretcast Yes, this patch is available under the new license. Thank you, @apaprocki. We will be splitting this into updated and more granular patches. CHANGE

[PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

apaprocki added a comment. @hubert.reinterpretcast Yes, this patch is available under the new license. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D18360/new/ https://reviews.llvm.org/D18360 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

hubert.reinterpretcast added a comment. Herald added subscribers: jdoerfert, arphaman. @apaprocki, as mentioned in our recent RFC ( http://lists.llvm.org/pipermail/llvm-dev/2019-February/130175.html ), IBM is working on AIX support for Clang and LLVM. We would like to continue the work on this

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

WuZhao added a comment. I wan to point to some issues, because I am also doing AIX support based on your patch and I tested and verified. Firstly, in the lib/Driver/ToolChains.cpp line 3756 case llvm::Triple::ppc64: addPathIfExists(D, getDriver().SysRoot + getDriver().Dir + "/../lib64",

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

majnemer accepted this revision. majnemer added a reviewer: majnemer. majnemer added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D18360 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://list

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

apaprocki updated this revision to Diff 64518. apaprocki added a comment. Increased context and removed accidental inclusion of Solaris change. https://reviews.llvm.org/D18360 Files: lib/Basic/Targets.cpp lib/Driver/Driver.cpp lib/Driver/ToolChains.cpp lib/Driver/ToolChains.h lib/Driv

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

majnemer added a comment. Could you please attach a diff with additional context? Comment at: lib/Driver/ToolChains.cpp:3717-3735 @@ -3678,1 +3716,21 @@ + // We need a detected GCC installation on Linux to provide libstdc++'s + // headers. We handled the libc++ case above. +

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

apaprocki updated this revision to Diff 64376. https://reviews.llvm.org/D18360 Files: lib/Basic/Targets.cpp lib/Driver/Driver.cpp lib/Driver/ToolChains.cpp lib/Driver/ToolChains.h lib/Driver/Tools.cpp lib/Driver/Tools.h tools/libclang/CIndexer.cpp Index: tools/libclang/CIndexer.cpp

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

apaprocki added inline comments. Comment at: lib/Basic/Targets.cpp:718 @@ +717,3 @@ +Builder.defineMacro("_LONG_LONG"); +Builder.defineMacro("_ALL_SOURCE"); +Builder.defineMacro("_REENTRANT"); majnemer wrote: > apaprocki wrote: > > majnemer wrote: > >

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

majnemer added inline comments. Comment at: lib/Basic/Targets.cpp:718 @@ +717,3 @@ +Builder.defineMacro("_LONG_LONG"); +Builder.defineMacro("_ALL_SOURCE"); +Builder.defineMacro("_REENTRANT"); apaprocki wrote: > majnemer wrote: > > Are we really suppose

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

apaprocki added inline comments. Comment at: lib/Basic/Targets.cpp:718 @@ +717,3 @@ +Builder.defineMacro("_LONG_LONG"); +Builder.defineMacro("_ALL_SOURCE"); +Builder.defineMacro("_REENTRANT"); majnemer wrote: > Are we really supposed to define this mac

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

majnemer added a subscriber: majnemer. Comment at: lib/Basic/Targets.cpp:718 @@ +717,3 @@ +Builder.defineMacro("_LONG_LONG"); +Builder.defineMacro("_ALL_SOURCE"); +Builder.defineMacro("_REENTRANT"); Are we really supposed to define this macro? Does GC

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

apaprocki added a comment. (I also re-based the patch on top of `trunk`) https://reviews.llvm.org/D18360 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

apaprocki updated this revision to Diff 64328. apaprocki added a comment. Fixed `wchar_t` type. https://reviews.llvm.org/D18360 Files: lib/Basic/Targets.cpp lib/Driver/Driver.cpp lib/Driver/ToolChains.cpp lib/Driver/ToolChains.h lib/Driver/Tools.cpp lib/Driver/Tools.h tools/libcla

Re: [PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

WuZhao added a subscriber: WuZhao. WuZhao added a comment. Hi , I find one mistake in the lib/Basic/Targets.cpp. On my AIX 7.1 machine /usr/include/sys/inttypes.h, 64 bits wchar_t is unsigned int, not signed int. #ifndef _WCHAR_T #define _WCHAR_T #ifdef __64BIT__ typedef unsigned int

[PATCH] D18360: Add AIX Target/ToolChain to Clang Driver

apaprocki created this revision. apaprocki added a subscriber: cfe-commits. This patch adds Clang driver support for the AIX platform. This allows Clang to be used for compiling code / checking for errors, but does not allow for building executables, as AIX uses XCOFF and not ELF. After applyi