From 8886138438a7f30bfd70ae962b84ffd5a88e79eb Mon Sep 17 00:00:00 2001 From: Michael Gottesman Date: Mon, 16 Mar 2015 07:02:30 +0000 Subject: [PATCH] [objc-arc] Rename ConnectTDBUTraversals => PairUpRetainsReleases. This is a name that is more descriptive of what the method really does. NFC. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@232349 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/ObjCARC/ObjCARCOpts.cpp | 30 +++++++++++++------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/lib/Transforms/ObjCARC/ObjCARCOpts.cpp b/lib/Transforms/ObjCARC/ObjCARCOpts.cpp index 5a73bc378f7..31c5666b202 100644 --- a/lib/Transforms/ObjCARC/ObjCARCOpts.cpp +++ b/lib/Transforms/ObjCARC/ObjCARCOpts.cpp @@ -459,15 +459,16 @@ namespace { DenseMap &Releases, SmallVectorImpl &DeadInsts, Module *M); - bool ConnectTDBUTraversals(DenseMap &BBStates, - BlotMapVector &Retains, - DenseMap &Releases, Module *M, - SmallVectorImpl &NewRetains, - SmallVectorImpl &NewReleases, - SmallVectorImpl &DeadInsts, - RRInfo &RetainsToMove, RRInfo &ReleasesToMove, - Value *Arg, bool KnownSafe, - bool &AnyPairsCompletelyEliminated); + bool + PairUpRetainsAndReleases(DenseMap &BBStates, + BlotMapVector &Retains, + DenseMap &Releases, Module *M, + SmallVectorImpl &NewRetains, + SmallVectorImpl &NewReleases, + SmallVectorImpl &DeadInsts, + RRInfo &RetainsToMove, RRInfo &ReleasesToMove, + Value *Arg, bool KnownSafe, + bool &AnyPairsCompletelyEliminated); bool PerformCodePlacement(DenseMap &BBStates, BlotMapVector &Retains, @@ -1453,7 +1454,7 @@ void ObjCARCOpt::MoveCalls(Value *Arg, RRInfo &RetainsToMove, } -bool ObjCARCOpt::ConnectTDBUTraversals( +bool ObjCARCOpt::PairUpRetainsAndReleases( DenseMap &BBStates, BlotMapVector &Retains, DenseMap &Releases, Module *M, @@ -1705,11 +1706,10 @@ bool ObjCARCOpt::PerformCodePlacement( // Connect the dots between the top-down-collected RetainsToMove and // bottom-up-collected ReleasesToMove to form sets of related calls. NewRetains.push_back(Retain); - bool PerformMoveCalls = - ConnectTDBUTraversals(BBStates, Retains, Releases, M, NewRetains, - NewReleases, DeadInsts, RetainsToMove, - ReleasesToMove, Arg, KnownSafe, - AnyPairsCompletelyEliminated); + bool PerformMoveCalls = PairUpRetainsAndReleases( + BBStates, Retains, Releases, M, NewRetains, NewReleases, DeadInsts, + RetainsToMove, ReleasesToMove, Arg, KnownSafe, + AnyPairsCompletelyEliminated); if (PerformMoveCalls) { // Ok, everything checks out and we're all set. Let's move/delete some