Skip to content

Comments

Convert deprecated functions to use modern atomic variable handling#806

Draft
SiliconA-Z wants to merge 1 commit intoswiftlang:mainfrom
SiliconA-Z:atomics-2
Draft

Convert deprecated functions to use modern atomic variable handling#806
SiliconA-Z wants to merge 1 commit intoswiftlang:mainfrom
SiliconA-Z:atomics-2

Conversation

@SiliconA-Z
Copy link
Contributor

Replaced deprecated atomic functions with the recommended C11 equivalents.

@SiliconA-Z
Copy link
Contributor Author

@compnerd Thoughts on this?

@SiliconA-Z SiliconA-Z force-pushed the atomics-2 branch 3 times, most recently from 19c218b to b323122 Compare December 16, 2023 00:04
@compnerd
Copy link
Member

I don't see a good reason for any of the changes in src/allocator.c. Switching the Windows path ... I might want to think a bit more on, but I'm not sure that we should change the non-Windows path.

Replaced deprecated atomic functions with the recommended C11 equivalents.
@SiliconA-Z
Copy link
Contributor Author

@compnerd I just unified the paths so we don't have to worry anymore

@SiliconA-Z SiliconA-Z marked this pull request as draft December 16, 2023 16:16
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