diff options
author | Dan Gohman <gohman@apple.com> | 2011-12-12 18:16:56 +0000 |
---|---|---|
committer | Dan Gohman <gohman@apple.com> | 2011-12-12 18:16:56 +0000 |
commit | 28588ff7aa417eded80101ea8d7fe4c7dcd8a1c1 (patch) | |
tree | 438f8aefbeacb7e7c61485be231f6a3acdb6cc00 | |
parent | dbf350a5a8f5fb5321a4e11e381f0023fbe61d15 (diff) | |
download | llvm-28588ff7aa417eded80101ea8d7fe4c7dcd8a1c1.tar.gz llvm-28588ff7aa417eded80101ea8d7fe4c7dcd8a1c1.tar.bz2 llvm-28588ff7aa417eded80101ea8d7fe4c7dcd8a1c1.tar.xz |
Inline SetSeqToRelease into its only caller, since it's more clear that way.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@146383 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Transforms/Scalar/ObjCARC.cpp | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/lib/Transforms/Scalar/ObjCARC.cpp b/lib/Transforms/Scalar/ObjCARC.cpp index 5444534cbf..2288f391ce 100644 --- a/lib/Transforms/Scalar/ObjCARC.cpp +++ b/lib/Transforms/Scalar/ObjCARC.cpp @@ -1251,16 +1251,6 @@ namespace { Seq = NewSeq; } - void SetSeqToRelease(MDNode *M) { - if (Seq == S_None || Seq == S_Use) { - Seq = M ? S_MovableRelease : S_Release; - RRI.ReleaseMetadata = M; - } else if (Seq != S_MovableRelease || RRI.ReleaseMetadata != M) { - Seq = S_Release; - RRI.ReleaseMetadata = 0; - } - } - Sequence GetSeq() const { return Seq; } @@ -2352,8 +2342,11 @@ ObjCARCOpt::VisitBottomUp(BasicBlock *BB, if (S.GetSeq() == S_Release || S.GetSeq() == S_MovableRelease) NestingDetected = true; - S.SetSeqToRelease(Inst->getMetadata(ImpreciseReleaseMDKind)); S.RRI.clear(); + + MDNode *ReleaseMetadata = Inst->getMetadata(ImpreciseReleaseMDKind); + S.SetSeq(ReleaseMetadata ? S_MovableRelease : S_Release); + S.RRI.ReleaseMetadata = ReleaseMetadata; S.RRI.KnownSafe = S.IsKnownNested() || S.IsKnownIncremented(); S.RRI.IsTailCallRelease = cast<CallInst>(Inst)->isTailCall(); S.RRI.Calls.insert(Inst); |