Skip to content

Conversation

@nate-chandler
Copy link

@nate-chandler nate-chandler commented Oct 22, 2025

To enable allocator implementations that store context in a frame header.

Deallocation has an address to be deallocated, not a size.
@nate-chandler nate-chandler force-pushed the general-coro/20251009/1 branch from ddd60f2 to ce05a0a Compare October 22, 2025 23:09
@nate-chandler nate-chandler force-pushed the general-coro/20251009/1 branch from ce05a0a to 4e8c83c Compare October 23, 2025 00:50
@nate-chandler nate-chandler changed the title [Coro] RetconOnceDynamic: Distinguish frame allocs and pass frame pointers. stable/21.x: [Coro] RetconOnceDynamic: Distinguish frame allocs and pass frame pointers. Oct 23, 2025
Copy link

@aschwaighofer aschwaighofer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Can you add a test that uses the llvm.coro.alloca.alloc/free.frame intrinsics?

Doing so enables implementations to allocate extra space in a header.
Doing so enables allocator implementation to use the frame to store
contextual info.
Mark the allocator parameter passed to the allocation functions
swiftcoro.
@nate-chandler nate-chandler force-pushed the general-coro/20251009/1 branch from 4e8c83c to a1c6568 Compare October 23, 2025 18:30
@nate-chandler
Copy link
Author

Fixed.

@nate-chandler
Copy link
Author

@swift-ci test llvm

@nate-chandler
Copy link
Author

swiftlang/swift#85055

@swift-ci test

@nate-chandler nate-chandler merged commit e52b2dd into swiftlang:stable/21.x Oct 24, 2025
5 checks passed
@nate-chandler nate-chandler deleted the general-coro/20251009/1 branch October 24, 2025 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants