diff options
| author | James Molloy <james.molloy@arm.com> | 2012-05-04 10:55:22 +0000 |
|---|---|---|
| committer | James Molloy <james.molloy@arm.com> | 2012-05-04 10:55:22 +0000 |
| commit | 392da48160bd92ceb486792780467cbfdb2d0e8c (patch) | |
| tree | 17b215252bcc1cfd72072422d0ebb1677fa94da3 /lib/Sema/SemaChecking.cpp | |
| parent | 2e4fd6d9a1c8ba9c400324d230cfc49050550dea (diff) | |
Fix handling of wint_t - we can't assume wint_t is purely an integer promotion of wchar_t - they may differ in signedness.
Teach ASTContext about WIntType, and have it taken from TargetInfo like WCharType. Should fix test/Sema/format-strings.c for ARM, with the exception of one subtest which will fail if wint_t and wchar_t are the same size and wint_t is signed, wchar_t is unsigned.
There'll be a followup commit to fix that.
Reviewed by Chandler and Hans at http://llvm.org/reviews/r/8
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@156165 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaChecking.cpp')
| -rw-r--r-- | lib/Sema/SemaChecking.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/Sema/SemaChecking.cpp b/lib/Sema/SemaChecking.cpp index dad0827488..0c8309e58e 100644 --- a/lib/Sema/SemaChecking.cpp +++ b/lib/Sema/SemaChecking.cpp @@ -2392,7 +2392,8 @@ CheckPrintfHandler::HandlePrintfSpecifier(const analyze_printf::PrintfSpecifier // or 'short' to an 'int'. This is done because printf is a varargs // function. if (const ImplicitCastExpr *ICE = dyn_cast<ImplicitCastExpr>(Ex)) - if (ICE->getType() == S.Context.IntTy) { + if (ICE->getType() == S.Context.IntTy || + ICE->getType() == S.Context.UnsignedIntTy) { // All further checking is done on the subexpression. Ex = ICE->getSubExpr(); if (ATR.matchesType(S.Context, Ex->getType())) |