[ownership] Change BorrowedValue::visitInteriorPointerOperand to pass IntPtrOperand as a value, not a const ref.

1. The reason to do this is that an InteriorPointerOperand is a value type.
2. Sometimes one wants to be able to use the IntPtrOperand that is not a const &
   and the value type formulation always works.

Functionality wise, this is NFCI.

Author: gottesmm

include/swift/SIL/OwnershipUtils.h

@@ -533,7 +533,7 @@ struct BorrowedValue {
   /// we were able to understand all uses and thus guarantee we found all
   /// interior pointer uses. Returns false otherwise.
   bool visitInteriorPointerOperands(
-      function_ref<void(const InteriorPointerOperand &)> func) const;
+      function_ref<void(InteriorPointerOperand)> func) const;
 
   /// Visit all immediate uses of this borrowed value and if any of them are
   /// reborrows, place them in BorrowingOperand form into \p

lib/SIL/Utils/OwnershipUtils.cpp

@@ -601,7 +601,7 @@ bool BorrowedValue::visitExtendedLocalScopeEndingUses(
 }
 
 bool BorrowedValue::visitInteriorPointerOperands(
-    function_ref<void(const InteriorPointerOperand &)> func) const {
+    function_ref<void(InteriorPointerOperand)> func) const {
   SmallVector<Operand *, 32> worklist(value->getUses());
   while (!worklist.empty()) {
     auto *op = worklist.pop_back_val();