amyk updated this revision to Diff 376723. amyk added a comment. Addressed comment made by @daltenty. Update the patch to account set the wchar type to `unsigned short` only on 32-bit AIX.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110428/new/ https://reviews.llvm.org/D110428 Files: clang/test/Sema/wchar.c Index: clang/test/Sema/wchar.c =================================================================== --- clang/test/Sema/wchar.c +++ clang/test/Sema/wchar.c @@ -4,7 +4,8 @@ typedef __WCHAR_TYPE__ wchar_t; #if defined(_WIN32) || defined(_M_IX86) || defined(__CYGWIN__) \ - || defined(_M_X64) || defined(__ORBIS__) || defined(SHORT_WCHAR) + || defined(_M_X64) || defined(__ORBIS__) || defined(SHORT_WCHAR) \ + || (defined(_AIX) && !defined(__64BIT__)) #define WCHAR_T_TYPE unsigned short #elif defined(__aarch64__) // See AArch64TargetInfo constructor -- unsigned on non-darwin non-OpenBSD non-NetBSD.
Index: clang/test/Sema/wchar.c =================================================================== --- clang/test/Sema/wchar.c +++ clang/test/Sema/wchar.c @@ -4,7 +4,8 @@ typedef __WCHAR_TYPE__ wchar_t; #if defined(_WIN32) || defined(_M_IX86) || defined(__CYGWIN__) \ - || defined(_M_X64) || defined(__ORBIS__) || defined(SHORT_WCHAR) + || defined(_M_X64) || defined(__ORBIS__) || defined(SHORT_WCHAR) \ + || (defined(_AIX) && !defined(__64BIT__)) #define WCHAR_T_TYPE unsigned short #elif defined(__aarch64__) // See AArch64TargetInfo constructor -- unsigned on non-darwin non-OpenBSD non-NetBSD.
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
