Files
swift-mirror/test/SILOptimizer/sil_combine_moveonly.sil
Erik Eckstein 1bcfa3acc1 SILCombine: don't sink mark_dependence
Don't risk sinking mark_dependence out of the lifetime of the base value.
2026-02-17 17:06:04 +01:00

232 lines
8.0 KiB
Plaintext

// RUN: %target-sil-opt -sil-print-types -enable-experimental-feature MoveOnlyEnumDeinits -enable-experimental-feature Lifetimes -enable-sil-verify-all -sil-combine %s | %FileCheck %s
// REQUIRES: swift_feature_MoveOnlyEnumDeinits
// REQUIRES: swift_feature_Lifetimes
sil_stage canonical
import Builtin
import Swift
struct S : ~Copyable {
deinit {}
}
struct S2 : ~Copyable {
@_hasStorage var a: Int { get set }
@_hasStorage var b: Int { get set }
}
struct FileDescriptor: ~Copyable {
var fd: Builtin.Int64
init(fd: Builtin.Int64)
deinit
}
enum MaybeFileDescriptor: ~Copyable {
case some(FileDescriptor)
case nothing
consuming func discard() { discard self }
deinit
}
struct Wrapper<T>: ~Copyable {
var t: T
}
struct NonCopyableForwardingStruct : ~Copyable {
var some: Int
deinit
}
enum NonCopyableForwardingEnum : ~Copyable {
case some(Int)
case nothing
deinit
}
sil @getWrappedValue : $@convention(thin) <T> (@in_guaranteed Wrapper<T>) -> @out T
// Test that a release_value is not removed for a struct-with-deinit.
// Doing so would forget the deinit.
//
// public func testStructDeinit() {
// var s = S()
// }
//
// CHECK-LABEL: sil @testStructDeinit : $@convention(thin) () -> () {
// CHECK: release_value %{{.*}} : $S
// CHECK-LABEL: } // end sil function 'testStructDeinit'
sil @testStructDeinit : $@convention(thin) () -> () {
bb0:
%0 = struct $S ()
release_value %0 : $S
%5 = tuple ()
return %5 : $()
}
// Test that a release_value is not replaced for a enum-with-deinit.
// Doing so would forget the deinit.
//
// CHECK-LABEL: sil hidden [noinline] @testEnumDeinit : $@convention(thin) () -> () {
// CHECK: release_value %{{.*}} : $MaybeFileDescriptor
// CHECK-LABEL: } // end sil function 'testEnumDeinit'
sil hidden [noinline] @testEnumDeinit : $@convention(thin) () -> () {
bb0:
%0 = integer_literal $Builtin.Int64, 0
%26 = struct $FileDescriptor (%0 : $Builtin.Int64)
%38 = enum $MaybeFileDescriptor, #MaybeFileDescriptor.some!enumelt, %26 : $FileDescriptor
release_value %38 : $MaybeFileDescriptor
%42 = tuple ()
return %42 : $()
}
// Test that a release of a trivial payload is not removed.
//
// CHECK-LABEL: sil hidden [noinline] @testEnumDeinitTrivialPayload : $@convention(thin) () -> () {
// CHECK: release_value %{{.*}} : $MaybeFileDescriptor
// CHECK-LABEL: } // end sil function 'testEnumDeinitTrivialPayload'
sil hidden [noinline] @testEnumDeinitTrivialPayload : $@convention(thin) () -> () {
bb0:
%0 = enum $MaybeFileDescriptor, #MaybeFileDescriptor.nothing!enumelt // users: %2, %1
debug_value %0 : $MaybeFileDescriptor, var, name "maybe" // id: %1
release_value %0 : $MaybeFileDescriptor
%9 = tuple ()
return %9 : $()
}
// Test that a move into a discarded value does not result in a destroy_addr
//
// CHECK-LABEL: sil @testNoDeinit : $@convention(method) <T> (@in Wrapper<T>) -> @out T {
// CHECK-NOT: destroy_addr
// CHECK-LABEL: } // end sil function 'testNoDeinit'
sil @testNoDeinit : $@convention(method) <T> (@in Wrapper<T>) -> @out T {
bb0(%0 : $*T, %1 : $*Wrapper<T>):
%2 = function_ref @getWrappedValue : $@convention(thin) <T> (@in_guaranteed Wrapper<T>) -> @out T
%3 = apply %2<T>(%0, %1) : $@convention(thin) <τ_0_0> (@in_guaranteed Wrapper<τ_0_0>) -> @out τ_0_0
%4 = alloc_stack $Wrapper<T>
copy_addr [take] %1 to [init] %4 : $*Wrapper<T>
// discard
dealloc_stack %4 : $*Wrapper<T>
%9 = tuple ()
return %9 : $()
}
// CHECK-LABEL: sil hidden [ossa] @testSDeinit : $@convention(method) (@owned S) -> () {
// CHECK: bb0(%0 : @owned $S):
// CHECK: [[DD:%.*]] = drop_deinit %0 : $S
// CHECK: destroy_value [[DD]] : $S
// CHECK-LABEL: } // end sil function 'testSDeinit'
sil hidden [ossa] @testSDeinit : $@convention(method) (@owned S) -> () {
bb0(%0 : @owned $S):
%1 = drop_deinit %0 : $S
destroy_value %1 : $S
%64 = tuple ()
return %64 : $()
}
// Test a "dead" non-Copyable struct-with-deinit that forwards a
// single trivial value. The destroy_value cannot be eliminated
// without first devirtualizing the deinit.
//
// CHECK-LABEL: sil [ossa] @testNonCopyableForwardingStructDeinit : $@convention(thin) (Int) -> () {
// CHECK: [[STRUCT:%[0-9]+]] = struct $NonCopyableForwardingStruct (%0 : $Int)
// CHECK: destroy_value [[STRUCT]]
// CHECK-LABEL: } // end sil function 'testNonCopyableForwardingStructDeinit'
sil [ossa] @testNonCopyableForwardingStructDeinit : $@convention(thin) (Int) -> () {
bb0(%0: $Int):
%1 = struct $NonCopyableForwardingStruct(%0 : $Int)
destroy_value %1
%13 = tuple ()
return %13
}
// Test a "dead" non-Copyable enum-with-deinit that forwards a single
// trivial value. The destroy_value cannot be eliminated without first
// devirtualizing the deinit.
//
// CHECK-LABEL: sil [ossa] @testNonCopyableForwardingEnumDeinit : $@convention(thin) (Int) -> () {
// CHECK: [[ENUM:%[0-9]+]] = enum $NonCopyableForwardingEnum, #NonCopyableForwardingEnum.some!enumelt, %0
// CHECK: destroy_value [[ENUM]]
// CHECK-LABEL: } // end sil function 'testNonCopyableForwardingEnumDeinit'
sil [ossa] @testNonCopyableForwardingEnumDeinit : $@convention(thin) (Int) -> () {
bb0(%0 : $Int):
%1 = enum $NonCopyableForwardingEnum, #NonCopyableForwardingEnum.some!enumelt, %0
destroy_value %1
%13 = tuple ()
return %13
}
// Test a "dead" non-Copyable enum-with-deinit and an empty case. The
// destroy_value cannot be eliminated without first devirtualizing the
// deinit.
//
// CHECK-LABEL: sil [ossa] @testNonCopyableEmptyEnumDeinit : $@convention(thin) () -> () {
// CHECK: [[ENUM:%[0-9]+]] = enum $NonCopyableForwardingEnum, #NonCopyableForwardingEnum.nothing
// CHECK: destroy_value [[ENUM]]
// CHECK-LABEL: } // end sil function 'testNonCopyableEmptyEnumDeinit'
sil [ossa] @testNonCopyableEmptyEnumDeinit : $@convention(thin) () -> () {
bb0:
%0 = enum $NonCopyableForwardingEnum, #NonCopyableForwardingEnum.nothing
destroy_value %0
%13 = tuple ()
return %13
}
// CHECK-LABEL: sil [ossa] @dont_remove_destructure_struct_of_non_copyable :
// CHECK: destructure_struct
// CHECK-LABEL: } // end sil function 'dont_remove_destructure_struct_of_non_copyable'
sil [ossa] @dont_remove_destructure_struct_of_non_copyable : $@convention(method) (@owned S2) -> () {
bb0(%0 : @owned $S2):
(%1, %2) = destructure_struct %0
%r = tuple ()
return %r
}
// CHECK-LABEL: sil [ossa] @dont_fold_destroy_of_take_enum_with_deinit_ossa :
// CHECK: bb0(%0 : $*MaybeFileDescriptor):
// CHECK-NEXT: %1 = unchecked_take_enum_data_addr %0
// CHECK-NEXT: destroy_addr %1
// CHECK: } // end sil function 'dont_fold_destroy_of_take_enum_with_deinit_ossa'
sil [ossa] @dont_fold_destroy_of_take_enum_with_deinit_ossa : $@convention(thin) (@in MaybeFileDescriptor) -> () {
bb0(%0 : $*MaybeFileDescriptor):
%1 = unchecked_take_enum_data_addr %0, #MaybeFileDescriptor.some!enumelt
destroy_addr %1
%r = tuple ()
return %r : $()
}
// CHECK-LABEL: sil @dont_fold_destroy_of_take_enum_with_deinit :
// CHECK: bb0(%0 : $*MaybeFileDescriptor):
// CHECK-NEXT: %1 = unchecked_take_enum_data_addr %0
// CHECK-NEXT: destroy_addr %1
// CHECK: } // end sil function 'dont_fold_destroy_of_take_enum_with_deinit'
sil @dont_fold_destroy_of_take_enum_with_deinit : $@convention(thin) (@in MaybeFileDescriptor) -> () {
bb0(%0 : $*MaybeFileDescriptor):
%1 = unchecked_take_enum_data_addr %0, #MaybeFileDescriptor.some!enumelt
destroy_addr %1
%r = tuple ()
return %r : $()
}
// CHECK-LABEL: sil [ossa] @dont_move_mark_dependence_out_of_scope :
// CHECK: mark_dependence
// CHECK-NEXT: end_borrow
// CHECK: } // end sil function 'dont_move_mark_dependence_out_of_scope'
sil [ossa] @dont_move_mark_dependence_out_of_scope : $@convention(method) <Element where Element : ~Copyable> (Int, @lifetime(copy 1) @inout Span<Element>) -> @lifetime(borrow 1) @owned Span<Element> {
bb0(%0 : $Int, %1 : $*Span<Element>):
%2 = load_borrow %1
%3 = struct_extract %2, #Span._pointer
%4 = struct $Span<Element> (%3, %0), forwarding: @owned
%5 = mark_dependence [nonescaping] %4 on %2
end_borrow %2
br bb1
bb1:
return %5
}