Remove BorrowingOperand.getImplicitUses.

It is buggy/misleading because of dead-def handling. The current operand may be
re-pushed if it is a dead borrow scope.

Author: atrick

include/swift/SIL/OwnershipUtils.h

@@ -460,12 +460,6 @@ struct BorrowingOperand {
   /// Return the borrowing operand's value.
   SILValue getScopeIntroducingUserResult();
 
-  /// Compute the implicit uses that this borrowing operand "injects" into the
-  /// set of its operands uses.
-  ///
-  /// E.x.: end_apply uses.
-  void getImplicitUses(SmallVectorImpl<Operand *> &foundUses) const;
-
   void print(llvm::raw_ostream &os) const;
   SWIFT_DEBUG_DUMP { print(llvm::dbgs()); }
 

lib/SIL/Utils/OwnershipUtils.cpp

@@ -828,20 +828,6 @@ SILValue BorrowingOperand::getScopeIntroducingUserResult() {
   llvm_unreachable("covered switch");
 }
 
-void BorrowingOperand::getImplicitUses(
-    SmallVectorImpl<Operand *> &foundUses) const {
-  // FIXME: this visitScopeEndingUses should never return false once dead
-  // borrows are disallowed.
-  auto handleUse = [&](Operand *endOp) {
-    foundUses.push_back(endOp);
-    return true;
-  };
-  if (!visitScopeEndingUses(handleUse, handleUse)) {
-    // Special-case for dead borrows.
-    foundUses.push_back(op);
-  }
-}
-
 //===----------------------------------------------------------------------===//
 //                             Borrow Introducers
 //===----------------------------------------------------------------------===//