diff options
| author | Fariborz Jahanian <fjahanian@apple.com> | 2011-02-25 18:38:59 +0000 |
|---|---|---|
| committer | Fariborz Jahanian <fjahanian@apple.com> | 2011-02-25 18:38:59 +0000 |
| commit | bff158dc6788ca804bea975422aea674dd2ea48a (patch) | |
| tree | 5ae95ec015a6a99d2ffa3bd9a211824d22dfd3f4 | |
| parent | a6ce608b97d6443f8e47f3cb20354be7b0cdeb43 (diff) | |
| download | bcm5719-llvm-bff158dc6788ca804bea975422aea674dd2ea48a.tar.gz bcm5719-llvm-bff158dc6788ca804bea975422aea674dd2ea48a.zip | |
Print optional message for attr(unavailable) in C++ mode.
// rdar://9046492
llvm-svn: 126499
| -rw-r--r-- | clang/include/clang/Basic/DiagnosticSemaKinds.td | 2 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaOverload.cpp | 23 | ||||
| -rw-r--r-- | clang/test/SemaCXX/attr-unavailable.cpp | 11 |
3 files changed, 31 insertions, 5 deletions
diff --git a/clang/include/clang/Basic/DiagnosticSemaKinds.td b/clang/include/clang/Basic/DiagnosticSemaKinds.td index 487b3c4d6a5..3c00e8b5c3b 100644 --- a/clang/include/clang/Basic/DiagnosticSemaKinds.td +++ b/clang/include/clang/Basic/DiagnosticSemaKinds.td @@ -1312,6 +1312,8 @@ def err_ovl_ambiguous_call : Error< "call to %0 is ambiguous">; def err_ovl_deleted_call : Error< "call to %select{unavailable|deleted}0 function %1">; +def err_ovl_unavailable_call : Error< + "call to unavailable function %0: %1">; def err_ovl_ambiguous_member_call : Error< "call to member function %0 is ambiguous">; def err_ovl_deleted_member_call : Error< diff --git a/clang/lib/Sema/SemaOverload.cpp b/clang/lib/Sema/SemaOverload.cpp index c59bfec8116..1da95b6a358 100644 --- a/clang/lib/Sema/SemaOverload.cpp +++ b/clang/lib/Sema/SemaOverload.cpp @@ -7709,11 +7709,24 @@ Sema::BuildOverloadedCallExpr(Scope *S, Expr *Fn, UnresolvedLookupExpr *ULE, break; case OR_Deleted: - Diag(Fn->getSourceRange().getBegin(), diag::err_ovl_deleted_call) - << Best->Function->isDeleted() - << ULE->getName() - << Fn->getSourceRange(); - CandidateSet.NoteCandidates(*this, OCD_AllCandidates, Args, NumArgs); + { + llvm::StringRef Message; + if (const UnavailableAttr *UA = + Best->Function->getAttr<UnavailableAttr>()) + Message = UA->getMessage(); + + if (Message.empty()) + Diag(Fn->getSourceRange().getBegin(), diag::err_ovl_deleted_call) + << Best->Function->isDeleted() + << ULE->getName() + << Fn->getSourceRange(); + else + Diag(Fn->getSourceRange().getBegin(), diag::err_ovl_unavailable_call) + << ULE->getName() + << Message + << Fn->getSourceRange(); + CandidateSet.NoteCandidates(*this, OCD_AllCandidates, Args, NumArgs); + } break; } diff --git a/clang/test/SemaCXX/attr-unavailable.cpp b/clang/test/SemaCXX/attr-unavailable.cpp index 6f5aa5e78c4..81db505e1e2 100644 --- a/clang/test/SemaCXX/attr-unavailable.cpp +++ b/clang/test/SemaCXX/attr-unavailable.cpp @@ -18,3 +18,14 @@ void test_foo(short* sp) { int &(*fp3)(int) = foo; void (*fp4)(...) = foo; // expected-error{{'foo' is unavailable}} } + +namespace radar9046492 { +// rdar://9046492 +#define FOO __attribute__((unavailable("not available - replaced"))) + +void foo() FOO; // expected-note {{candidate function has been explicitly made unavailable}} + +void bar() { + foo(); // expected-error {{call to unavailable function 'foo': not available - replaced}} +} +} |

