[llvm] [Transforms][IPO] Add func suffix in ArgumentPromotion and DeadArgume… (PR #105742)

via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 29 19:18:50 PDT 2024


================
@@ -876,6 +876,8 @@ bool DeadArgumentEliminationPass::removeDeadStuffFromFunction(Function *F) {
   // it again.
   F->getParent()->getFunctionList().insert(F->getIterator(), NF);
   NF->takeName(F);
+  if (NumArgumentsEliminated)
----------------
yonghong-song wrote:

> What cases are we excluding with the "if" here? If we don't change the signature, we don't recreate the function in the first place.

Okay, uploaded new revision to introduce ".retelim" suffix is the function sole change is to remove return values.

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


More information about the llvm-commits mailing list