[llvm] [FuncAttrs] Relax norecurse attribute inference (PR #139943)

[David Sherwood via llvm-commits]

================
@@ -2071,24 +2072,36 @@ static void addNoRecurseAttrs(const SCCNodeSet &SCCNodes,
   if (!F || !F->hasExactDefinition() || F->doesNotRecurse())
     return;
 
-  // If all of the calls in F are identifiable and are to norecurse functions, F
-  // is norecurse. This check also detects self-recursion as F is not currently
-  // marked norecurse, so any called from F to F will not be marked norecurse.
-  for (auto &BB : *F)
-    for (auto &I : BB.instructionsWithoutDebug())
+  if (F->hasAddressTaken())
----------------
david-arm wrote:

It also may not add much value because you're only going one level deeper.

https://github.com/llvm/llvm-project/pull/139943