diff options
| author | Hans Wennborg <hans@hanshq.net> | 2012-05-08 17:21:31 +0000 |
|---|---|---|
| committer | Hans Wennborg <hans@hanshq.net> | 2012-05-08 17:21:31 +0000 |
| commit | 101d4e0c8ffbcdaaa58cddf1c20f98aa1b4501c4 (patch) | |
| tree | 151fa3df415b3ecb77378869993a7a078f4eaa53 /lib/Analysis/FormatString.cpp | |
| parent | fd04a6a88255ce3a90e5f73aef69c08bb5a35677 (diff) | |
Make -Wformat accept printf("%hhx", c); with -funsigned-char
For "%hhx", printf expects an unsigned char. This makes Clang
accept a 'char' argument for that also when using -funsigned-char.
This fixes PR12761.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@156388 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Analysis/FormatString.cpp')
| -rw-r--r-- | lib/Analysis/FormatString.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/Analysis/FormatString.cpp b/lib/Analysis/FormatString.cpp index ab69c06911..f776c89cc5 100644 --- a/lib/Analysis/FormatString.cpp +++ b/lib/Analysis/FormatString.cpp @@ -265,10 +265,9 @@ bool ArgTypeResult::matchesType(ASTContext &C, QualType argTy) const { break; case BuiltinType::Char_S: case BuiltinType::SChar: - return T == C.UnsignedCharTy; case BuiltinType::Char_U: case BuiltinType::UChar: - return T == C.SignedCharTy; + return T == C.UnsignedCharTy || T == C.SignedCharTy; case BuiltinType::Short: return T == C.UnsignedShortTy; case BuiltinType::UShort: |