Skip to content

Commit 01c2e0f

Browse files
committed
[irgen] Make AsyncLetBegin and AsyncLetFinish UNKNOWN_MEMEFFECTS instead of ArgMemOnly.
In a discussion with @rjmccall, we agreed that these should really be UNKNOWN_MEMEFFECTS so we are conservative. Just slicing this off from a larger patch stream.
1 parent dc7f30f commit 01c2e0f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

include/swift/Runtime/RuntimeFunctions.def

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2570,7 +2570,7 @@ FUNCTION(AsyncLetBegin,
25702570
),
25712571
ATTRS(NoUnwind),
25722572
EFFECT(RuntimeEffect::Concurrency),
2573-
MEMEFFECTS(ArgMemOnly))
2573+
UNKNOWN_MEMEFFECTS)
25742574

25752575
/// void swift_asyncLet_finish(
25762576
/// AsyncLet *alet,
@@ -2592,7 +2592,7 @@ FUNCTION(AsyncLetFinish,
25922592
),
25932593
ATTRS(NoUnwind),
25942594
EFFECT(RuntimeEffect::Concurrency),
2595-
MEMEFFECTS(ArgMemOnly))
2595+
UNKNOWN_MEMEFFECTS)
25962596

25972597
/// void swift_task_run_inline(
25982598
/// OpaqueValue *result,

0 commit comments

Comments
 (0)