Skip to content

Commit 3c4275a

Browse files
committed
[CanonicalizeOSSALifetime] Renamed member.
The member just clears the values. After it adopted BitfieldRef, the call to invalidate on the liveness instance was already superfluous.
1 parent dfe2c00 commit 3c4275a

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

include/swift/SILOptimizer/Utils/CanonicalizeOSSALifetime.h

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -325,10 +325,9 @@ class CanonicalizeOSSALifetime final {
325325
liveness->initializeDef(getCurrentDef());
326326
}
327327

328-
void invalidateLiveness() {
328+
void clear() {
329329
consumingBlocks.clear();
330330
debugValues.clear();
331-
liveness->invalidate();
332331
discoveredBlocks.clear();
333332
}
334333

lib/SILOptimizer/Utils/CanonicalizeOSSALifetime.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1108,7 +1108,7 @@ bool CanonicalizeOSSALifetime::computeLiveness() {
11081108
// Step 1: compute liveness
11091109
if (!computeCanonicalLiveness()) {
11101110
LLVM_DEBUG(llvm::errs() << "Failed to compute canonical liveness?!\n");
1111-
invalidateLiveness();
1111+
clear();
11121112
return false;
11131113
}
11141114
if (currentDef->isLexical()) {
@@ -1145,7 +1145,7 @@ void CanonicalizeOSSALifetime::rewriteLifetimes() {
11451145
// Step 6: rewrite copies and delete extra destroys
11461146
rewriteCopies();
11471147

1148-
invalidateLiveness();
1148+
clear();
11491149
consumes.clear();
11501150
}
11511151

0 commit comments

Comments
 (0)