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

Alex Voicu via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 10 06:23:24 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);
----------------
AlexVlx wrote:

I did not intend to alter behaviour here, but merely to adopt the changed interface. There's no easy way to use she LangAS::Default -> Target mapping here in this helper function, hence the hardcoding to 0.

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


More information about the cfe-commits mailing list