SIL: rename updateBorrowedFrom to updateBorrowArguments

eeckstein · eeckstein · commit 6b8c6a3c3b0e · 2024-11-12T09:26:58.000+01:00
NFC
diff --git a/SwiftCompilerSources/Sources/Optimizer/PassManager/PassRegistration.swift b/SwiftCompilerSources/Sources/Optimizer/PassManager/PassRegistration.swift
@@ -136,5 +136,5 @@ private func registerSwiftAnalyses() {
 
 private func registerUtilities() {
   registerVerifier()
-  registerBorrowedFromUpdater()
+  registerBorrowArgumentsUpdater()
 }
diff --git a/SwiftCompilerSources/Sources/Optimizer/Utilities/BorrowArgumentsUpdater.swift b/SwiftCompilerSources/Sources/Optimizer/Utilities/BorrowArgumentsUpdater.swift
@@ -1,4 +1,4 @@
-//===--- BorrowedFromUpdater.swift ----------------------------------------===//
+//===--- BorrowArgumentsUpdater.swift -------------------------------------===//
 //
 // This source file is part of the Swift.org open source project
 //
@@ -98,8 +98,8 @@ private func addEnclosingValues(
   return true
 }
 
-func registerBorrowedFromUpdater() {
-  BridgedUtilities.registerBorrowedFromUpdater(
+func registerBorrowArgumentsUpdater() {
+  BridgedUtilities.registerBorrowArgumentsUpdater(
     { (bridgedCtxt: BridgedPassContext, bridgedFunction: BridgedFunction) in
       let context = FunctionPassContext(_bridged: bridgedCtxt)
       let function = bridgedFunction.function;
diff --git a/SwiftCompilerSources/Sources/Optimizer/Utilities/CMakeLists.txt b/SwiftCompilerSources/Sources/Optimizer/Utilities/CMakeLists.txt
@@ -9,7 +9,7 @@
 swift_compiler_sources(Optimizer
   AccessUtilsTest.swift
   AddressUtils.swift
-  BorrowedFromUpdater.swift
+  BorrowArgumentsUpdater.swift
   BorrowUtils.swift
   SpecializationCloner.swift
   DiagnosticEngine.swift
diff --git a/include/swift/SILOptimizer/OptimizerBridging.h b/include/swift/SILOptimizer/OptimizerBridging.h
@@ -130,12 +130,12 @@ struct BridgedPostDomTree {
 
 struct BridgedUtilities {
   typedef void (* _Nonnull VerifyFunctionFn)(BridgedPassContext, BridgedFunction);
-  typedef void (* _Nonnull UpdateBorrowedFromFn)(BridgedPassContext, BridgedFunction);
-  typedef void (* _Nonnull UpdateBorrowedFromPhisFn)(BridgedPassContext, BridgedArrayRef);
+  typedef void (* _Nonnull UpdateFunctionFn)(BridgedPassContext, BridgedFunction);
+  typedef void (* _Nonnull UpdatePhisFn)(BridgedPassContext, BridgedArrayRef);
 
   static void registerVerifier(VerifyFunctionFn verifyFunctionFn);
-  static void registerBorrowedFromUpdater(UpdateBorrowedFromFn updateBorrowedFromFn,
-                                          UpdateBorrowedFromPhisFn updateBorrowedFromPhisFn);
+  static void registerBorrowArgumentsUpdater(UpdateFunctionFn updateBorrowedFromFn,
+                                             UpdatePhisFn updateBorrowedFromPhisFn);
 };
 
 struct BridgedBasicBlockSet {
diff --git a/include/swift/SILOptimizer/Utils/OwnershipOptUtils.h b/include/swift/SILOptimizer/Utils/OwnershipOptUtils.h
@@ -357,9 +357,9 @@ bool extendStoreBorrow(StoreBorrowInst *sbi,
                        DeadEndBlocks *deadEndBlocks,
                        InstModCallbacks callbacks = InstModCallbacks());
 
-void updateBorrowedFrom(SILPassManager *pm, SILFunction *f);
+void updateAllBorrowArguments(SILPassManager *pm, SILFunction *f);
 
-void updateBorrowedFromPhis(SILPassManager *pm, ArrayRef<SILPhiArgument *> phis);
+void updateBorrowArguments(SILPassManager *pm, ArrayRef<SILPhiArgument *> phis);
 
 } // namespace swift
 
diff --git a/lib/SILOptimizer/LoopTransforms/ArrayPropertyOpt.cpp b/lib/SILOptimizer/LoopTransforms/ArrayPropertyOpt.cpp
@@ -837,7 +837,7 @@ class SwiftArrayPropertyOptPass : public SILFunctionTransform {
       if (getFunction()->getModule().getOptions().VerifyAll)
         getFunction()->verifyCriticalEdges();
 
-      updateBorrowedFrom(getPassManager(), Fn);
+      updateAllBorrowArguments(getPassManager(), Fn);
 
       // We preserve the dominator tree. Let's invalidate everything
       // else.
diff --git a/lib/SILOptimizer/LoopTransforms/LoopRotate.cpp b/lib/SILOptimizer/LoopTransforms/LoopRotate.cpp
@@ -514,7 +514,7 @@ class LoopRotation : public SILFunctionTransform {
     }
 
     if (changed) {
-      updateBorrowedFrom(PM, f);
+      updateAllBorrowArguments(PM, f);
       // We preserve loop info and the dominator tree.
       domAnalysis->lockInvalidation();
       loopAnalysis->lockInvalidation();
diff --git a/lib/SILOptimizer/Mandatory/ClosureLifetimeFixup.cpp b/lib/SILOptimizer/Mandatory/ClosureLifetimeFixup.cpp
@@ -1529,7 +1529,7 @@ class ClosureLifetimeFixup : public SILFunctionTransform {
     if (fixupClosureLifetimes(*getFunction(), dominanceAnalysis,
                               deadEndBlocksAnalysis, checkStackNesting,
                               modifiedCFG)) {
-      updateBorrowedFrom(getPassManager(), getFunction());
+      updateAllBorrowArguments(getPassManager(), getFunction());
       if (checkStackNesting){
         modifiedCFG |=
           StackNesting::fixNesting(getFunction()) == StackNesting::Changes::CFG;
diff --git a/lib/SILOptimizer/SILCombiner/SILCombine.cpp b/lib/SILOptimizer/SILCombiner/SILCombine.cpp
@@ -620,7 +620,7 @@ class SILCombine : public SILFunctionTransform {
     bool Changed = Combiner.runOnFunction(*getFunction());
 
     if (Changed) {
-      updateBorrowedFrom(getPassManager(), getFunction());
+      updateAllBorrowArguments(getPassManager(), getFunction());
       // Invalidate everything.
       invalidateAnalysis(SILAnalysis::InvalidationKind::FunctionBody);
     }
diff --git a/lib/SILOptimizer/SemanticARC/OwnedToGuaranteedPhiOpt.cpp b/lib/SILOptimizer/SemanticARC/OwnedToGuaranteedPhiOpt.cpp
@@ -268,7 +268,7 @@ bool swift::semanticarc::tryConvertOwnedPhisToGuaranteedPhis(Context &ctx) {
   }
 
   if (madeChange)
-    updateBorrowedFrom(ctx.pm, &ctx.fn);
+    updateAllBorrowArguments(ctx.pm, &ctx.fn);
 
   return madeChange;
 }
diff --git a/lib/SILOptimizer/SemanticARC/SemanticARCOpts.cpp b/lib/SILOptimizer/SemanticARC/SemanticARCOpts.cpp
@@ -177,7 +177,8 @@ struct SemanticARCOpts : SILFunctionTransform {
     // converted to guaranteed, ignoring the phi, try convert those phis to be
     // guaranteed.
     if (tryConvertOwnedPhisToGuaranteedPhis(visitor.ctx)) {
-      updateBorrowedFrom(getPassManager(), &f);
+      updateAllBorrowArguments(getPassManager(), &f);
+
       // We return here early to save a little compile time so we do not
       // invalidate analyses redundantly.
       return invalidateAnalysis(
@@ -186,7 +187,7 @@ struct SemanticARCOpts : SILFunctionTransform {
 
     // Otherwise, we only deleted instructions and did not touch phis.
     if (didEliminateARCInsts) {
-      updateBorrowedFrom(getPassManager(), &f);
+      updateAllBorrowArguments(getPassManager(), &f);
       invalidateAnalysis(SILAnalysis::InvalidationKind::Instructions);
     }
   }
diff --git a/lib/SILOptimizer/Transforms/ConditionForwarding.cpp b/lib/SILOptimizer/Transforms/ConditionForwarding.cpp
@@ -117,7 +117,7 @@ class ConditionForwarding : public SILFunctionTransform {
       }
     }
     if (Changed) {
-      updateBorrowedFrom(getPassManager(), F);
+      updateAllBorrowArguments(getPassManager(), F);
       invalidateAnalysis(SILAnalysis::InvalidationKind::BranchesAndInstructions);
     }
   }
diff --git a/lib/SILOptimizer/Transforms/CopyPropagation.cpp b/lib/SILOptimizer/Transforms/CopyPropagation.cpp
@@ -635,7 +635,7 @@ void CopyPropagation::run() {
 
   // Invalidate analyses.
   if (changed || deleter.hadCallbackInvocation()) {
-    updateBorrowedFrom(getPassManager(), getFunction());
+    updateAllBorrowArguments(getPassManager(), getFunction());
     // Preserves NonLocalAccessBlockAnalysis.
     accessBlockAnalysis->lockInvalidation();
     invalidateAnalysis(SILAnalysis::InvalidationKind::Instructions);
diff --git a/lib/SILOptimizer/Transforms/PerformanceInliner.cpp b/lib/SILOptimizer/Transforms/PerformanceInliner.cpp
@@ -1286,7 +1286,7 @@ bool SILPerformanceInliner::inlineCallsIntoFunction(SILFunction *Caller) {
   if (invalidatedStackNesting) {
     StackNesting::fixNesting(Caller);
   }
-  updateBorrowedFrom(pm, Caller);
+  updateAllBorrowArguments(pm, Caller);
 
   // If we were asked to verify our caller after inlining all callees we could
   // find into it, do so now. This makes it easier to catch verification bugs in
diff --git a/lib/SILOptimizer/Transforms/SILMem2Reg.cpp b/lib/SILOptimizer/Transforms/SILMem2Reg.cpp
@@ -2297,7 +2297,7 @@ class SILMem2Reg : public SILFunctionTransform {
                                         accessBlockAnalysis, calleeAnalysis)
                           .run();
     if (madeChange) {
-      updateBorrowedFrom(getPassManager(), f);
+      updateAllBorrowArguments(getPassManager(), f);
       invalidateAnalysis(SILAnalysis::InvalidationKind::Instructions);
     }
   }
diff --git a/lib/SILOptimizer/Transforms/SimplifyCFG.cpp b/lib/SILOptimizer/Transforms/SimplifyCFG.cpp
@@ -2189,7 +2189,7 @@ bool SimplifyCFG::simplifySwitchEnumBlock(SwitchEnumInst *SEI) {
     }
     Builder.createBranch(loc, LiveBlock, PayLoad);
     SEI->eraseFromParent();
-    updateBorrowedFromPhis(PM, { cast<SILPhiArgument>(LiveBlock->getArgument(0)) });
+    updateBorrowArguments(PM, { cast<SILPhiArgument>(LiveBlock->getArgument(0)) });
   } else {
     if (SEI->getOperand()->getOwnershipKind() == OwnershipKind::Owned) {
       Builder.createDestroyValue(loc, SEI->getOperand());
@@ -3784,7 +3784,7 @@ bool SimplifyCFG::simplifyArgument(SILBasicBlock *BB, unsigned i) {
 
   proj->eraseFromParent();
 
-  updateBorrowedFromPhis(PM, { NewArg });
+  updateBorrowArguments(PM, { NewArg });
 
   return true;
 }
diff --git a/lib/SILOptimizer/Utils/BasicBlockOptUtils.cpp b/lib/SILOptimizer/Utils/BasicBlockOptUtils.cpp
@@ -169,7 +169,7 @@ void BasicBlockCloner::updateSSAAfterCloning() {
       }
     }
   }
-  updateBorrowedFromPhis(pm, updateSSAPhis);
+  updateBorrowArguments(pm, updateSSAPhis);
 }
 
 void BasicBlockCloner::sinkAddressProjections() {
diff --git a/lib/SILOptimizer/Utils/InstOptUtils.cpp b/lib/SILOptimizer/Utils/InstOptUtils.cpp
@@ -687,7 +687,7 @@ swift::castValueToABICompatibleType(SILBuilder *builder, SILPassManager *pm,
     builder->createBranch(loc, contBB, {noneValue});
     builder->setInsertionPoint(contBB->begin());
 
-    updateBorrowedFromPhis(pm, { phi });
+    updateBorrowArguments(pm, { phi });
 
     return {lookThroughBorrowedFromUser(phi), true};
   }
diff --git a/lib/SILOptimizer/Utils/OwnershipOptUtils.cpp b/lib/SILOptimizer/Utils/OwnershipOptUtils.cpp
@@ -1913,27 +1913,27 @@ bool swift::extendStoreBorrow(StoreBorrowInst *sbi,
 //                            Swift Bridging
 //===----------------------------------------------------------------------===//
 
-static BridgedUtilities::UpdateBorrowedFromFn updateBorrowedFromFunction;
-static BridgedUtilities::UpdateBorrowedFromPhisFn updateBorrowedFromPhisFunction;
+static BridgedUtilities::UpdateFunctionFn updateAllBorrowArgumentsFunction;
+static BridgedUtilities::UpdatePhisFn updateBorrowArgumentsFunction;
 
-void BridgedUtilities::registerBorrowedFromUpdater(UpdateBorrowedFromFn updateBorrowedFromFn,
-                                                   UpdateBorrowedFromPhisFn updateBorrowedFromPhisFn) {
-  updateBorrowedFromFunction = updateBorrowedFromFn;
-  updateBorrowedFromPhisFunction = updateBorrowedFromPhisFn;
+void BridgedUtilities::registerBorrowArgumentsUpdater(UpdateFunctionFn updateAllBorrowArgumentsFn,
+                                                      UpdatePhisFn updateBorrowArgumentsFn) {
+  updateAllBorrowArgumentsFunction = updateAllBorrowArgumentsFn;
+  updateBorrowArgumentsFunction = updateBorrowArgumentsFn;
 }
 
-void swift::updateBorrowedFrom(SILPassManager *pm, SILFunction *f) {
-  if (updateBorrowedFromFunction)
-    updateBorrowedFromFunction({pm->getSwiftPassInvocation()}, {f});
+void swift::updateAllBorrowArguments(SILPassManager *pm, SILFunction *f) {
+  if (updateAllBorrowArgumentsFunction)
+    updateAllBorrowArgumentsFunction({pm->getSwiftPassInvocation()}, {f});
 }
 
-void swift::updateBorrowedFromPhis(SILPassManager *pm, ArrayRef<SILPhiArgument *> phis) {
-  if (!updateBorrowedFromPhisFunction)
+void swift::updateBorrowArguments(SILPassManager *pm, ArrayRef<SILPhiArgument *> phis) {
+  if (!updateBorrowArgumentsFunction)
     return;
 
   llvm::SmallVector<BridgedValue, 8> bridgedPhis;
   for (SILPhiArgument *phi : phis) {
     bridgedPhis.push_back({phi});
   }
-  updateBorrowedFromPhisFunction({pm->getSwiftPassInvocation()}, ArrayRef(bridgedPhis));
+  updateBorrowArgumentsFunction({pm->getSwiftPassInvocation()}, ArrayRef(bridgedPhis));
 }

Original file line number	Diff line number	Diff line change
`@@ -136,5 +136,5 @@ private func registerSwiftAnalyses() {`
`136`	`136`
`137`	`137`	`private func registerUtilities() {`
`138`	`138`	`registerVerifier()`
`139`		`- registerBorrowedFromUpdater()`
	`139`	`+ registerBorrowArgumentsUpdater()`
`140`	`140`	`}`
Original file line number	Diff line number	Diff line change
`@@ -514,7 +514,7 @@ class LoopRotation : public SILFunctionTransform {`
`514`	`514`	`}`
`515`	`515`
`516`	`516`	`if (changed) {`
`517`		`- updateBorrowedFrom(PM, f);`
	`517`	`+ updateAllBorrowArguments(PM, f);`
`518`	`518`	`// We preserve loop info and the dominator tree.`
`519`	`519`	`domAnalysis->lockInvalidation();`
`520`	`520`	`loopAnalysis->lockInvalidation();`
Original file line number	Diff line number	Diff line change
`@@ -620,7 +620,7 @@ class SILCombine : public SILFunctionTransform {`
`620`	`620`	`bool Changed = Combiner.runOnFunction(*getFunction());`
`621`	`621`
`622`	`622`	`if (Changed) {`
`623`		`- updateBorrowedFrom(getPassManager(), getFunction());`
	`623`	`+ updateAllBorrowArguments(getPassManager(), getFunction());`
`624`	`624`	`// Invalidate everything.`
`625`	`625`	`invalidateAnalysis(SILAnalysis::InvalidationKind::FunctionBody);`
`626`	`626`	`}`
Original file line number	Diff line number	Diff line change
`@@ -268,7 +268,7 @@ bool swift::semanticarc::tryConvertOwnedPhisToGuaranteedPhis(Context &ctx) {`
`268`	`268`	`}`
`269`	`269`
`270`	`270`	`if (madeChange)`
`271`		`- updateBorrowedFrom(ctx.pm, &ctx.fn);`
	`271`	`+ updateAllBorrowArguments(ctx.pm, &ctx.fn);`
`272`	`272`
`273`	`273`	`return madeChange;`
`274`	`274`	`}`