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 | 967b9cec23dc8723bd417685ab2cbcb72d533067 (patch) | |
| tree | fcffe9fe0eb23a416c26e76fbc28cacd172b9703 /clang/lib/Analysis/FormatString.cpp | |
| parent | 24ac479a7d0e583bb39295baf494f49cc9f1dac3 (diff) | |
| download | bcm5719-llvm-967b9cec23dc8723bd417685ab2cbcb72d533067.tar.gz bcm5719-llvm-967b9cec23dc8723bd417685ab2cbcb72d533067.zip | |
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.
llvm-svn: 156388
Diffstat (limited to 'clang/lib/Analysis/FormatString.cpp')
| -rw-r--r-- | clang/lib/Analysis/FormatString.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/clang/lib/Analysis/FormatString.cpp b/clang/lib/Analysis/FormatString.cpp index ab69c069114..f776c89cc56 100644 --- a/clang/lib/Analysis/FormatString.cpp +++ b/clang/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: |
