[OperandOwnership] @in_constant operands consume.

Match the behavior that MemoryLifetimeVerifier::setFuncOperandBits
verifies for addresses for values.

Author: nate-chandler

lib/SIL/IR/OperandOwnership.cpp

@@ -468,6 +468,7 @@ static OperandOwnership getFunctionArgOwnership(SILArgumentConvention argConv,
 
   switch (argConv) {
   case SILArgumentConvention::Indirect_In:
+  case SILArgumentConvention::Indirect_In_Constant:
   case SILArgumentConvention::Direct_Owned:
     return OperandOwnership::ForwardingConsume;
 
@@ -478,7 +479,6 @@ static OperandOwnership getFunctionArgOwnership(SILArgumentConvention argConv,
   // borrow scope in the caller. In contrast, a begin_apply /does/ have an
   // explicit borrow scope in the caller so we must treat arguments passed to it
   // as being borrowed for the entire region of coroutine execution.
-  case SILArgumentConvention::Indirect_In_Constant:
   case SILArgumentConvention::Indirect_In_Guaranteed:
   case SILArgumentConvention::Direct_Guaranteed:
     // For an apply that begins a borrow scope, its arguments are borrowed