Recursive functions should be marked when used from another function. Fixes http...

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@112045 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Sema/SemaExpr.cpp b/lib/Sema/SemaExpr.cpp
index c8e8ecda861a9a112ff16e013f4a8b612cd5ca29..6c25c272f38e357f9987c65e069049cf643017c8 100644 (file)
--- a/lib/Sema/SemaExpr.cpp
+++ b/lib/Sema/SemaExpr.cpp
@@ -7683,7 +7683,10 @@ void Sema::MarkDeclarationReferenced(SourceLocation Loc, Decl *D) {
     }
 
     // FIXME: keep track of references to static functions
-    Function->setUsed(true);
+
+    // Recursive functions should be marked when used from another function.
+    if (CurContext != Function)
+      Function->setUsed(true);
 
     return;
   }

diff --git a/test/Sema/warn-unused-function.c b/test/Sema/warn-unused-function.c
index 5ae0cce0798d3c17b7ec6f882d6cefb1c91b0287..24d4fade9b31334a32981421c529bab28fc0326d 100644 (file)
--- a/test/Sema/warn-unused-function.c
+++ b/test/Sema/warn-unused-function.c
@@ -44,3 +44,6 @@ static void f11(void) { }  // expected-warning{{unused}}
 
 static void f12(void) { }  // expected-warning{{unused}}
 static void f12(void);
+
+// PR7923
+static void unused(void) { unused(); }  // expected-warning{{unused}}