Skip to content

Commit ec367ce

Browse files
committed
Drop overridable entry point for swift_asyncLet_start.
This entry point is a small shim over `swift_task_create_common`, which is overridable. We don't need a separate override point here.
1 parent 819615e commit ec367ce

File tree

3 files changed

+5
-21
lines changed

3 files changed

+5
-21
lines changed

stdlib/public/CompatibilityOverride/CompatibilityOverrideConcurrency.def

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -159,18 +159,6 @@ OVERRIDE_TASK(task_asyncMainDrainQueue, void,
159159
SWIFT_EXPORT_FROM(swift_Concurrency), SWIFT_CC(swift), swift::,
160160
, )
161161

162-
OVERRIDE_ASYNC_LET(asyncLet_start, void,
163-
SWIFT_EXPORT_FROM(swift_Concurrency), SWIFT_CC(swift),
164-
swift::,
165-
(AsyncLet *alet,
166-
TaskOptionRecord *options,
167-
const Metadata *futureResultType,
168-
void *closureEntryPoint,
169-
HeapObject *closureContext
170-
),
171-
(alet, options,
172-
futureResultType, closureEntryPoint, closureContext))
173-
174162
OVERRIDE_ASYNC_LET(asyncLet_wait, void, SWIFT_EXPORT_FROM(swift_Concurrency),
175163
SWIFT_CC(swiftasync), swift::,
176164
(OpaqueValue *result,

stdlib/public/Concurrency/AsyncLet.cpp

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -101,11 +101,11 @@ void swift::asyncLet_addImpl(AsyncTask *task, AsyncLet *asyncLet) {
101101
// ==== start ------------------------------------------------------------------
102102

103103
SWIFT_CC(swift)
104-
static void swift_asyncLet_startImpl(AsyncLet *alet,
105-
TaskOptionRecord *options,
106-
const Metadata *futureResultType,
107-
void *closureEntryPoint,
108-
HeapObject *closureContext) {
104+
void swift::swift_asyncLet_start(AsyncLet *alet,
105+
TaskOptionRecord *options,
106+
const Metadata *futureResultType,
107+
void *closureEntryPoint,
108+
HeapObject *closureContext) {
109109
auto flags = TaskCreateFlags();
110110
flags.setEnqueueJob(true);
111111

unittests/runtime/CompatibilityOverrideConcurrency.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -169,10 +169,6 @@ TEST_F(CompatibilityOverrideConcurrencyTest,
169169
swift_continuation_throwingResumeWithError(nullptr, nullptr);
170170
}
171171

172-
TEST_F(CompatibilityOverrideConcurrencyTest, test_swift_asyncLet_start) {
173-
swift_asyncLet_start(nullptr, nullptr, nullptr, nullptr, nullptr);
174-
}
175-
176172
TEST_F(CompatibilityOverrideConcurrencyTest, test_swift_asyncLet_wait) {
177173
swift_asyncLet_wait(nullptr, nullptr, nullptr, nullptr, nullptr);
178174
}

0 commit comments

Comments
 (0)