[clang] [llvm] [ARM] musttail fixes (PR #102896)

Eli Friedman via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 12 12:10:56 PDT 2024


================
@@ -5085,7 +5085,7 @@ RValue CodeGenFunction::EmitCall(const CGFunctionInfo &CallInfo,
   RawAddress SRetAlloca = RawAddress::invalid();
   llvm::Value *UnusedReturnSizePtr = nullptr;
   if (RetAI.isIndirect() || RetAI.isInAlloca() || RetAI.isCoerceAndExpand()) {
-    if (IsVirtualFunctionPointerThunk && RetAI.isIndirect()) {
+    if ((IsVirtualFunctionPointerThunk || IsMustTail) && RetAI.isIndirect()) {
----------------
efriedma-quic wrote:

Did you put this in by accident?  I can sort of see what you're getting at here, but it seems disconnected from the rest of the patch, and there aren't any tests.

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


More information about the cfe-commits mailing list