diff options
| author | Ted Kremenek <kremenek@apple.com> | 2010-02-01 23:23:50 +0000 |
|---|---|---|
| committer | Ted Kremenek <kremenek@apple.com> | 2010-02-01 23:23:50 +0000 |
| commit | 016b6052662c772ea8866694e54626e334214010 (patch) | |
| tree | bc491579c67fe384fcd8efcd4a17e38d26ef8a2e | |
| parent | 5b66b31774d8e5bb3cc43900d419362d8e93b398 (diff) | |
| download | bcm5719-llvm-016b6052662c772ea8866694e54626e334214010.tar.gz bcm5719-llvm-016b6052662c772ea8866694e54626e334214010.zip | |
Add format string type checking support for 'long double'.
llvm-svn: 95026
| -rw-r--r-- | clang/lib/Analysis/PrintfFormatString.cpp | 5 | ||||
| -rw-r--r-- | clang/test/Sema/format-strings.c | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/clang/lib/Analysis/PrintfFormatString.cpp b/clang/lib/Analysis/PrintfFormatString.cpp index d2bcbb04f9d..35c620adedc 100644 --- a/clang/lib/Analysis/PrintfFormatString.cpp +++ b/clang/lib/Analysis/PrintfFormatString.cpp @@ -311,8 +311,11 @@ ArgTypeResult FormatSpecifier::getArgType(ASTContext &Ctx) const { return ArgTypeResult(); } - if (CS.isDoubleArg()) + if (CS.isDoubleArg()) { + if (LM == AsLongDouble) + return Ctx.LongDoubleTy; return Ctx.DoubleTy; + } // FIXME: Handle other cases. return ArgTypeResult(); diff --git a/clang/test/Sema/format-strings.c b/clang/test/Sema/format-strings.c index a055bfe1822..d8f7e4f6931 100644 --- a/clang/test/Sema/format-strings.c +++ b/clang/test/Sema/format-strings.c @@ -167,6 +167,8 @@ void test10(int x, float f, int i, long long lli) { // This is fine, because there is an implicit conversion to an int. printf("%d", (unsigned char) 10); // no-warning printf("%d", (long long) 10); // expected-warning{{conversion specifies type 'int' but the argument has type 'long long'}} + printf("%Lf\n", (long double) 1.0); // no-warning + printf("%f\n", (long double) 1.0); // expected-warning{{conversion specifies type 'double' but the argument has type 'long double'}} } typedef struct __aslclient *aslclient; |

