diff options
| author | Fariborz Jahanian <fjahanian@apple.com> | 2013-03-14 23:09:00 +0000 |
|---|---|---|
| committer | Fariborz Jahanian <fjahanian@apple.com> | 2013-03-14 23:09:00 +0000 |
| commit | 95236b50ce51aff7cd1d3b3a3c5a8475cd93bcb2 (patch) | |
| tree | 6321310d3e983bdccb8ecd4603eb2b486ccdb5c8 /clang/lib/Sema/SemaDecl.cpp | |
| parent | 3e56dd4fc94a189505f80c09422470f81ce19ca4 (diff) | |
| download | bcm5719-llvm-95236b50ce51aff7cd1d3b3a3c5a8475cd93bcb2.tar.gz bcm5719-llvm-95236b50ce51aff7cd1d3b3a3c5a8475cd93bcb2.zip | |
Diagnose about extern "C" functions returning c++ objects
on first declaration only. // rdar://13364028
llvm-svn: 177127
Diffstat (limited to 'clang/lib/Sema/SemaDecl.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 4c6da45e5b3..245bc2c0e78 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -6793,7 +6793,8 @@ bool Sema::CheckFunctionDeclaration(Scope *S, FunctionDecl *NewFD, // If this function is declared as being extern "C", then check to see if // the function returns a UDT (class, struct, or union type) that is not C // compatible, and if it does, warn the user. - if (NewFD->isExternC()) { + // But, issue any diagnostic on the first declaration only. + if (NewFD->isExternC() && Previous.empty()) { QualType R = NewFD->getResultType(); if (R->isIncompleteType() && !R->isVoidType()) Diag(NewFD->getLocation(), diag::warn_return_value_udt_incomplete) |
