Merge pull request swiftlang#34637 from gottesmm/pr-027a025e7a95d70fcc7e98300987826a5948c8c4

[ownership] Move type dependent operand check out of the main visitor to eliminate another returning of an empty map from the visitor.

Author: gottesmm

lib/SIL/IR/OperandOwnership.cpp

@@ -570,9 +570,11 @@ OperandOwnershipKindClassifier::visitFullApply(FullApplySite apply) {
     return Map::allLive();
   }
 
-  // If we have a type dependent operand, return an empty map.
-  if (apply.getInstruction()->isTypeDependentOperand(op))
-    return Map();
+  // We should have early exited if we saw a type dependent operand, so we
+  // should never hit this.
+  //
+  // Lets just assert to be careful though.
+  assert(!apply.getInstruction()->isTypeDependentOperand(op));
 
   unsigned argIndex = apply.getCalleeArgIndex(op);
   auto conv = apply.getSubstCalleeConv();
@@ -972,6 +974,8 @@ OperandOwnershipKindClassifier::visitBuiltinInst(BuiltinInst *bi) {
 //===----------------------------------------------------------------------===//
 
 OperandOwnershipKindMap Operand::getOwnershipKindMap() const {
+  if (isTypeDependent())
+    return OperandOwnershipKindMap();
   OperandOwnershipKindClassifier classifier(getUser()->getModule(), *this);
   return classifier.visit(const_cast<SILInstruction *>(getUser()));
 }