Don't inline non-ossa function into an ossa function in the new mandatory performance opts

Update the new inliner with this check.
Inlining non-ossa into ossa can causes unexpected consequences.

Author: meg-gupta

SwiftCompilerSources/Sources/Optimizer/ModulePasses/MandatoryPerformanceOptimizations.swift

@@ -123,10 +123,17 @@ private func shouldInline(apply: FullApplySite, callee: Function, alreadyInlined
   if callee.isTransparent {
     return true
   }
+
+  if apply.parentFunction.hasOwnership && !callee.hasOwnership {
+    // Cannot inline a non-ossa function into an ossa function
+    return false
+  }
+
   if apply is BeginApplyInst {
     // Avoid co-routines because they might allocate (their context).
     return true
   }
+
   if apply.parentFunction.isGlobalInitOnceFunction && callee.inlineStrategy == .always {
     // Some arithmetic operations, like integer conversions, are not transparent but `inline(__always)`.
     // Force inlining them in global initializers so that it's possible to statically initialize the global.

lib/SILOptimizer/Utils/SILInliner.cpp

@@ -360,7 +360,8 @@ SILInliner::inlineFunction(SILFunction *calleeFunction, FullApplySite apply,
   PrettyStackTraceSILFunction callerTraceRAII("...into", apply.getFunction());
   assert(canInlineApplySite(apply)
          && "Asked to inline function that is unable to be inlined?!");
-
+  assert(!apply.getFunction()->hasOwnership() ||
+         calleeFunction->hasOwnership());
   SILInlineCloner cloner(calleeFunction, apply, FuncBuilder, IKind, ApplySubs,
                          deleter);
   cloner.cloneInline(appliedArgs);