Delete isValueAddressOrTrivial

If we want a short-circuit test for address types, that should be done
within getOwnershipKind() itself.

Author: atrick

include/swift/SIL/OwnershipUtils.h

@@ -63,9 +63,6 @@ class DeadEndBlocks;
 class MultiDefPrunedLiveness;
 struct BorrowedValue;
 
-/// Returns true if v is an address or trivial.
-bool isValueAddressOrTrivial(SILValue v);
-
 //===----------------------------------------------------------------------===//
 //                            Forwarding Utilities
 //

lib/SIL/Utils/OwnershipUtils.cpp

@@ -83,11 +83,6 @@ bool swift::hasPointerEscape(BorrowedValue value) {
   return false;
 }
 
-bool swift::isValueAddressOrTrivial(SILValue v) {
-  return v->getType().isAddress() ||
-         v->getOwnershipKind() == OwnershipKind::None;
-}
-
 bool swift::canOpcodeForwardGuaranteedValues(SILValue value) {
   // If we have an argument from a transforming terminator, we can forward
   // guaranteed.

lib/SIL/Verifier/SILOwnershipVerifier.cpp

@@ -579,7 +579,7 @@ bool SILValueOwnershipChecker::checkValueWithoutLifetimeEndingUses(
     }
   }
 
-  if (!isValueAddressOrTrivial(value)) {
+  if (value->getOwnershipKind() != OwnershipKind::None) {
     return !errorBuilder.handleMalformedSIL([&] {
       if (value->getOwnershipKind() == OwnershipKind::Owned) {
         llvm::errs() << "Error! Found a leaked owned value that was never "