Merge pull request #21765 from aschwaighofer/delete_dead_stripConvertFunctions

Author: swift-ci

include/swift/SIL/InstructionUtils.h

@@ -126,10 +126,6 @@ bool mayCheckRefCount(SILInstruction *User);
 /// run-time sanitizers.
 bool isSanitizerInstrumentation(SILInstruction *Instruction);
 
-/// If V is a convert_function or convert_escape_to_noescape return its operand
-/// recursively.
-SILValue stripConvertFunctions(SILValue V);
-
 /// Check that this is a partial apply of a reabstraction thunk and return the
 /// argument of the partial apply if it is.
 SILValue isPartialApplyOfReabstractionThunk(PartialApplyInst *PAI);

lib/SIL/InstructionUtils.cpp

@@ -312,22 +312,6 @@ bool swift::isSanitizerInstrumentation(SILInstruction *Instruction) {
   return false;
 }
 
-SILValue swift::stripConvertFunctions(SILValue V) {
-  while (true) {
-    if (auto CFI = dyn_cast<ConvertFunctionInst>(V)) {
-      V = CFI->getOperand();
-      continue;
-    }
-    else if (auto *Cvt = dyn_cast<ConvertEscapeToNoEscapeInst>(V)) {
-      V = Cvt->getOperand();
-      continue;
-    }
-    break;
-  }
-  return V;
-}
-
-
 SILValue swift::isPartialApplyOfReabstractionThunk(PartialApplyInst *PAI) {
   if (PAI->getNumArguments() != 1)
     return SILValue();

lib/SILOptimizer/Mandatory/DiagnoseStaticExclusivity.cpp

@@ -969,9 +969,6 @@ static void checkNoEscapePartialApplyUse(Operand *oper, FollowUse followUses) {
   }
 
   // Look through copies, borrows, and conversions.
-  //
-  // Note: This handles ConversionInst, which already includes everything in
-  // swift::stripConvertFunctions.
   if (SingleValueInstruction *copy = getSingleValueCopyOrCast(user)) {
     // Only follow the copied operand. Other operands are incidental,
     // as in the second operand of mark_dependence.