summaryrefslogtreecommitdiffstats
path: root/clang/lib/Sema/SemaDeclAttr.cpp
diff options
context:
space:
mode:
authorTed Kremenek <kremenek@apple.com>2009-08-14 20:51:58 +0000
committerTed Kremenek <kremenek@apple.com>2009-08-14 20:51:58 +0000
commit420d9a7a5e7d99450947a8ac27a889eada320460 (patch)
treefb41c28b994169b0c707248fcd38a19dc85af5db /clang/lib/Sema/SemaDeclAttr.cpp
parent527042b5a97168594d999e08cd1d42901e0ea6c4 (diff)
downloadbcm5719-llvm-420d9a7a5e7d99450947a8ac27a889eada320460.tar.gz
bcm5719-llvm-420d9a7a5e7d99450947a8ac27a889eada320460.zip
Use 'dyn_cast' instead of a check for a function declaration followed by a
'cast'. llvm-svn: 79041
Diffstat (limited to 'clang/lib/Sema/SemaDeclAttr.cpp')
-rw-r--r--clang/lib/Sema/SemaDeclAttr.cpp5
1 files changed, 3 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaDeclAttr.cpp b/clang/lib/Sema/SemaDeclAttr.cpp
index 4952e629001..e65b3aa6fdf 100644
--- a/clang/lib/Sema/SemaDeclAttr.cpp
+++ b/clang/lib/Sema/SemaDeclAttr.cpp
@@ -438,13 +438,14 @@ static void HandleMallocAttr(Decl *d, const AttributeList &Attr, Sema &S) {
return;
}
- if (!isFunction(d)) {
+ const FunctionDecl *FD = dyn_cast<FunctionDecl>(d);
+
+ if (!FD) {
S.Diag(Attr.getLoc(), diag::warn_attribute_wrong_decl_type)
<< Attr.getName() << 0 /*function*/;
return;
}
- const FunctionDecl *FD = cast<FunctionDecl>(d);
QualType RetTy = FD->getResultType();
if (!(RetTy->isAnyPointerType() || RetTy->isBlockPointerType())) {
OpenPOWER on IntegriCloud