[llvm] [FunctionAttrs] Remove unused legacy-PM runImpl template (NFC) (PR #202983)

[via llvm-commits]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-transforms

Author: Aditya Medhane (flash1729)

<details>
<summary>Changes</summary>

`runImpl` here is a leftover from the legacy pass manager (it takes `CallGraphSCC`), with no callers since the legacy PM was removed. It never instantiates, so it trips `-Wunused-template`. Removing the dead template.

NFC.

Part of #<!-- -->202945.

---
Full diff: https://github.com/llvm/llvm-project/pull/202983.diff


1 Files Affected:

- (modified) llvm/lib/Transforms/IPO/FunctionAttrs.cpp (-10) 


``````````diff
diff --git a/llvm/lib/Transforms/IPO/FunctionAttrs.cpp b/llvm/lib/Transforms/IPO/FunctionAttrs.cpp
index 467fab519d32a..d98d3a1c46309 100644
--- a/llvm/lib/Transforms/IPO/FunctionAttrs.cpp
+++ b/llvm/lib/Transforms/IPO/FunctionAttrs.cpp
@@ -2319,16 +2319,6 @@ void PostOrderFunctionAttrsPass::printPipeline(
     OS << "<skip-non-recursive-function-attrs>";
 }
 
-template <typename AARGetterT>
-static bool runImpl(CallGraphSCC &SCC, AARGetterT AARGetter) {
-  SmallVector<Function *, 8> Functions;
-  for (CallGraphNode *I : SCC) {
-    Functions.push_back(I->getFunction());
-  }
-
-  return !deriveAttrsInPostOrder(Functions, AARGetter).empty();
-}
-
 static bool addNoRecurseAttrsTopDown(Function &F) {
   if (F.doesNotRecurse())
     return false;

``````````

</details>


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