diff options
| author | Nick Lewycky <nicholas@mxc.ca> | 2011-12-02 23:21:43 +0000 |
|---|---|---|
| committer | Nick Lewycky <nicholas@mxc.ca> | 2011-12-02 23:21:43 +0000 |
| commit | 45ccba64ab34275cf272df01ae838524b94f5e2b (patch) | |
| tree | e96f326879cadd1e1c5e7c2c3bcdd909bbc81553 /clang/lib/Analysis/FormatString.cpp | |
| parent | b89f2a6e78bd9bf6e8e4484ecc379e9bb343df33 (diff) | |
| download | llvm-45ccba64ab34275cf272df01ae838524b94f5e2b.zip llvm-45ccba64ab34275cf272df01ae838524b94f5e2b.tar.gz llvm-45ccba64ab34275cf272df01ae838524b94f5e2b.tar.bz2 | |
Revert r145697 and dependent patch r145702. It added a dependency from
lib/Analysis to lib/Sema which is cyclical.
llvm-svn: 145724
Diffstat (limited to 'clang/lib/Analysis/FormatString.cpp')
| -rw-r--r-- | clang/lib/Analysis/FormatString.cpp | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/clang/lib/Analysis/FormatString.cpp b/clang/lib/Analysis/FormatString.cpp index 0171bb7..6498ded 100644 --- a/clang/lib/Analysis/FormatString.cpp +++ b/clang/lib/Analysis/FormatString.cpp @@ -230,8 +230,7 @@ bool ArgTypeResult::matchesType(ASTContext &C, QualType argTy) const { case SpecificTy: { argTy = C.getCanonicalType(argTy).getUnqualifiedType(); - QualType U = C.getCanonicalType(T); - if (U == argTy) + if (T == argTy) return true; // Check for "compatible types". if (const BuiltinType *BT = argTy->getAs<BuiltinType>()) @@ -240,26 +239,26 @@ bool ArgTypeResult::matchesType(ASTContext &C, QualType argTy) const { break; case BuiltinType::Char_S: case BuiltinType::SChar: - return U == C.UnsignedCharTy; + return T == C.UnsignedCharTy; case BuiltinType::Char_U: case BuiltinType::UChar: - return U == C.SignedCharTy; + return T == C.SignedCharTy; case BuiltinType::Short: - return U == C.UnsignedShortTy; + return T == C.UnsignedShortTy; case BuiltinType::UShort: - return U == C.ShortTy; + return T == C.ShortTy; case BuiltinType::Int: - return U == C.UnsignedIntTy; + return T == C.UnsignedIntTy; case BuiltinType::UInt: - return U == C.IntTy; + return T == C.IntTy; case BuiltinType::Long: - return U == C.UnsignedLongTy; + return T == C.UnsignedLongTy; case BuiltinType::ULong: - return U == C.LongTy; + return T == C.LongTy; case BuiltinType::LongLong: - return U == C.UnsignedLongLongTy; + return T == C.UnsignedLongLongTy; case BuiltinType::ULongLong: - return U == C.LongLongTy; + return T == C.LongLongTy; } return false; } @@ -486,3 +485,5 @@ bool FormatSpecifier::hasValidLengthModifier() const { } return false; } + + |