Remove @_unsafeNonescapableResult from unit tests.

Author: atrick

test/SILOptimizer/lifetime_dependence/lifetime_dependence_borrow_fail.swift

@@ -12,7 +12,6 @@ struct BV : ~Escapable {
   let p: UnsafeRawPointer
   let i: Int
 
-  @_unsafeNonescapableResult
   init(_ p: UnsafeRawPointer, _ i: Int) -> dependsOn(p) Self {
     self.p = p
     self.i = i
@@ -36,7 +35,6 @@ struct NE : ~Escapable {
   let p: UnsafeRawPointer
   let i: Int
 
-  @_unsafeNonescapableResult
   init(_ p: UnsafeRawPointer, _ i: Int) -> dependsOn(p) Self {
     self.p = p
     self.i = i

test/SILOptimizer/lifetime_dependence/lifetime_dependence_inherit.swift

@@ -8,6 +8,15 @@
 // REQUIRES: asserts
 // REQUIRES: swift_in_compiler
 
+// TODO: Remove @_unsafeNonescapableResult. Instead, the unsafe dependence should be expressed by a builtin that is
+// hidden within the function body.
+@_unsafeNonescapableResult
+func unsafeLifetime<T: ~Copyable & ~Escapable, U: ~Copyable & ~Escapable>(
+  dependent: consuming T, dependsOn source: borrowing U)
+  -> dependsOn(source) T {
+  dependent
+}
+
 struct BV : ~Escapable {
   let p: UnsafeRawPointer
   let i: Int
@@ -17,16 +26,16 @@ struct BV : ~Escapable {
     self.i = i
   }
 
-  @_unsafeNonescapableResult
-  init(independent p: UnsafeRawPointer, _ i: Int) {
+  init(independent p: UnsafeRawPointer, _ i: Int) -> dependsOn(p) Self {
     self.p = p
     self.i = i
   }
 
   consuming func derive() -> dependsOn(self) BV {
     // Technically, this "new" view does not depend on the 'view' argument.
     // This unsafely creates a new view with no dependence.
-    return BV(independent: self.p, self.i)
+    let bv = BV(independent: self.p, self.i)
+    return unsafeLifetime(dependent: bv, dependsOn: self)
   }
 }
 

test/SILOptimizer/lifetime_dependence/lifetime_dependence_inherit_fail.swift

@@ -8,6 +8,15 @@
 // REQUIRES: asserts
 // REQUIRES: swift_in_compiler
 
+// TODO: Remove @_unsafeNonescapableResult. Instead, the unsafe dependence should be expressed by a builtin that is
+// hidden within the function body.
+@_unsafeNonescapableResult
+func unsafeLifetime<T: ~Copyable & ~Escapable, U: ~Copyable & ~Escapable>(
+  dependent: consuming T, dependsOn source: borrowing U)
+  -> dependsOn(source) T {
+  dependent
+}
+
 struct BV : ~Escapable {
   let p: UnsafeRawPointer
   let i: Int
@@ -17,16 +26,16 @@ struct BV : ~Escapable {
     self.i = i
   }
 
-  @_unsafeNonescapableResult
-  init(independent p: UnsafeRawPointer, _ i: Int) {
+  init(independent p: UnsafeRawPointer, _ i: Int) -> dependsOn(p) Self {
     self.p = p
     self.i = i
   }
 
   consuming func derive() -> dependsOn(self) BV {
     // Technically, this "new" view does not depend on the 'view' argument.
     // This unsafely creates a new view with no dependence.
-    return BV(independent: self.p, self.i)
+    let bv = BV(independent: self.p, self.i)
+    return unsafeLifetime(dependent: bv, dependsOn: self)
   }
 }
 

test/SILOptimizer/lifetime_dependence/lifetime_dependence_optional.swift

@@ -10,9 +10,7 @@
 // Simply test that it is possible for a module to define a pseudo-Optional type without triggering any compiler errors.
 
 public protocol ExpressibleByNilLiteral: ~Copyable & ~Escapable {
-  // TODO: dependsOn(immortal)
-  @_unsafeNonescapableResult
-  init(nilLiteral: ())
+  init(nilLiteral: ()) -> dependsOn(immortal) Self
 }
 
 @frozen
@@ -30,10 +28,8 @@ extension Nillable: Sendable where Wrapped: ~Copyable & ~Escapable & Sendable {
 extension Nillable: BitwiseCopyable where Wrapped: BitwiseCopyable { }
 
 extension Nillable: ExpressibleByNilLiteral where Wrapped: ~Copyable & ~Escapable {
-  // TODO: dependsOn(immortal)
   @_transparent
-  @_unsafeNonescapableResult
-  public init(nilLiteral: ()) {
+  public init(nilLiteral: ()) -> dependsOn(immortal) Self {
     self = .none
   }
 }

test/SILOptimizer/lifetime_dependence/lifetime_dependence_param.swift

@@ -8,6 +8,15 @@
 // REQUIRES: asserts
 // REQUIRES: swift_in_compiler
 
+// TODO: Remove @_unsafeNonescapableResult. Instead, the unsafe dependence should be expressed by a builtin that is
+// hidden within the function body.
+@_unsafeNonescapableResult
+func unsafeLifetime<T: ~Copyable & ~Escapable, U: ~Copyable & ~Escapable>(
+  dependent: consuming T, dependsOn source: borrowing U)
+  -> dependsOn(source) T {
+  dependent
+}
+
 struct BV : ~Escapable {
   let p: UnsafeRawPointer
   let i: Int
@@ -17,8 +26,7 @@ struct BV : ~Escapable {
     self.i = i
   }
 
-  @_unsafeNonescapableResult
-  init(independent p: UnsafeRawPointer, _ i: Int) {
+  init(independent p: UnsafeRawPointer, _ i: Int) -> dependsOn(p) Self {
     self.p = p
     self.i = i
   }
@@ -30,7 +38,8 @@ struct BV : ~Escapable {
   consuming func derive() -> dependsOn(self) BV {
     // Technically, this "new" view does not depend on the 'view' argument.
     // This unsafely creates a new view with no dependence.
-    return BV(independent: self.p, self.i)
+    let bv = BV(independent: self.p, self.i)
+    return unsafeLifetime(dependent: bv, dependsOn: self)
   }
 }
 

test/SILOptimizer/lifetime_dependence/lifetime_dependence_scope_fixup.swift

@@ -17,8 +17,7 @@ struct NCContainer : ~Copyable {
 struct View : ~Escapable {
   let ptr: UnsafeRawBufferPointer
   let c: Int
-  @_unsafeNonescapableResult
-  init(_ ptr: UnsafeRawBufferPointer, _ c: Int) {
+  init(_ ptr: UnsafeRawBufferPointer, _ c: Int) -> dependsOn(p) Self {
     self.ptr = ptr
     self.c = c
   }

test/SILOptimizer/lifetime_dependence/lifetime_dependence_todo.swift

@@ -14,7 +14,6 @@ struct BV : ~Escapable {
   let p: UnsafeRawPointer
   let i: Int
 
-  @_unsafeNonescapableResult
   init(_ p: UnsafeRawPointer, _ i: Int) -> dependsOn(p) Self {
     self.p = p
     self.i = i