[clang] [clang][CodeGen] `sret` args should always point to the `alloca` AS, so use that (PR #114062)

John McCall via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 12 13:44:25 PST 2024


================
@@ -800,7 +800,9 @@ static ABIArgInfo classifyExpandedType(SwiftAggLowering &lowering,
   if (lowering.empty()) {
     return ABIArgInfo::getIgnore();
   } else if (lowering.shouldPassIndirectly(forReturn)) {
-    return ABIArgInfo::getIndirect(alignmentForIndirect, /*byval*/ false);
+    return ABIArgInfo::getIndirect(alignmentForIndirect,
+                                   /*AddrSpace*/ 0,
+                                   /*byval*/ false);
----------------
rjmccall wrote:

I'm asking you to change behavior in a way that's consistent with the way you're changing behavior in general in your patch.

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


More information about the cfe-commits mailing list