[move-only] Teach deinit devirtualization how to handle devirtualizing a deinit of a resilient type.

Specifically, such a type has an @in convention for its deinit and the pass was
setup expecting self to always be @owned.

rdar://109904633
(cherry picked from commit 1d04df9)

Author: gottesmm

include/swift/SIL/SILFunctionConventions.h

@@ -371,6 +371,11 @@ class SILFunctionConventions {
     return getNumIndirectSILResults();
   }
 
+  /// Returns the index of self.
+  unsigned getSILArgIndexOfSelf() const {
+    return getSILArgIndexOfFirstParam() + getNumParameters() - 1;
+  }
+
   /// Get the index into formal indirect results corresponding to the given SIL
   /// indirect result argument index.
   unsigned getIndirectFormalResultIndexForSILArg(unsigned argIdx) const {

lib/SILOptimizer/Mandatory/MoveOnlyDeinitInsertion.cpp

@@ -77,9 +77,21 @@ static bool performTransform(SILFunction &fn) {
           auto subMap =
               astType->getContextSubstitutionMap(nom->getModuleContext(), nom);
           SILBuilderWithScope builder(dvi);
+
+          SILValue value = dvi->getOperand();
+          auto conv = deinitFunc->getConventionsInContext();
+          if (conv.getSILArgumentConvention(conv.getSILArgIndexOfSelf())
+                  .isIndirectConvention()) {
+            auto *asi =
+                builder.createAllocStack(dvi->getLoc(), value->getType());
+            builder.emitStoreValueOperation(dvi->getLoc(), value, asi,
+                                            StoreOwnershipQualifier::Init);
+            value = asi;
+          }
           auto *funcRef = builder.createFunctionRef(dvi->getLoc(), deinitFunc);
-          builder.createApply(dvi->getLoc(), funcRef, subMap,
-                              dvi->getOperand());
+          builder.createApply(dvi->getLoc(), funcRef, subMap, value);
+          if (isa<AllocStackInst>(value))
+            builder.createDeallocStack(dvi->getLoc(), value);
           dvi->eraseFromParent();
           changed = true;
           continue;
@@ -103,9 +115,15 @@ static bool performTransform(SILFunction &fn) {
           auto *funcRef = builder.createFunctionRef(dai->getLoc(), deinitFunc);
           auto subMap = destroyType.getASTType()->getContextSubstitutionMap(
               nom->getModuleContext(), nom);
-          auto loadedValue = builder.emitLoadValueOperation(
-              dai->getLoc(), dai->getOperand(), LoadOwnershipQualifier::Take);
-          builder.createApply(dai->getLoc(), funcRef, subMap, loadedValue);
+
+          auto conv = deinitFunc->getConventionsInContext();
+          auto argConv =
+              conv.getSILArgumentConvention(conv.getSILArgIndexOfSelf());
+          SILValue value = dai->getOperand();
+          if (!argConv.isIndirectConvention())
+            value = builder.emitLoadValueOperation(
+                dai->getLoc(), dai->getOperand(), LoadOwnershipQualifier::Take);
+          builder.createApply(dai->getLoc(), funcRef, subMap, value);
           dai->eraseFromParent();
           changed = true;
           continue;