[llvm] [AArch64] Don't tail call memset if it would convert to a bzero. (PR #98969)

[Amara Emerson via llvm-commits]

================
@@ -685,7 +687,10 @@ bool llvm::returnTypeIsEligibleForTailCall(const Function *F,
          (IID == Intrinsic::memmove &&
           TLI.getLibcallName(RTLIB::MEMMOVE) == StringRef("memmove")) ||
          (IID == Intrinsic::memset &&
-          TLI.getLibcallName(RTLIB::MEMSET) == StringRef("memset"))) &&
+          TLI.getLibcallName(RTLIB::MEMSET) == StringRef("memset") &&
+          (!isa<ConstantInt>(Call->getOperand(1)) ||
+           !cast<ConstantInt>(Call->getOperand(1))->isZero() ||
+           !TLI.getLibcallName(RTLIB::BZERO)))) &&
----------------
aemerson wrote:

I think only the IR version tries to do the returns-first-arg trickery so we can probably just go with the first option.

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