Merge pull request #30073 from gottesmm/pr-2049c5667508c622325a30f14e18b0c78bb850bf

[ownership] Move TempRValueOpt before Semantic ARC Opts.

Author: gottesmm

lib/SILOptimizer/PassManager/PassPipeline.cpp

@@ -395,16 +395,21 @@ static void addPerfEarlyModulePassPipeline(SILPassPipelinePlan &P) {
   // we do not spend time optimizing them.
   P.addDeadFunctionElimination();
 
+  // Cleanup after SILGen: remove trivial copies to temporaries.
+  P.addTempRValueOpt();
+  // Cleanup after SILGen: remove unneeded borrows/copies.
   P.addSemanticARCOpts();
 
   // Strip ownership from non-transparent functions.
   if (P.getOptions().StripOwnershipAfterSerialization)
     P.addNonTransparentFunctionOwnershipModelEliminator();
 
-  // Start by cloning functions from stdlib.
+  // Start by linking in referenced functions from other modules.
   P.addPerformanceSILLinker();
 
-  // Cleanup after SILGen: remove trivial copies to temporaries.
+  // Cleanup after SILGen: remove trivial copies to temporaries. This version of
+  // temp-rvalue opt is here so that we can hit copies from non-ossa code that
+  // is linked in from the stdlib.
   P.addTempRValueOpt();
 
   // Add the outliner pass (Osize).