Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@DmitryOlshansky
Copy link

No description provided.

Comment on lines +13 to +17
version(X86_64) {
enum SYS_futex = 202;
} else version(AArch64) {
enum SYS_futex = 98;
}
Copy link

Choose a reason for hiding this comment

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

We should support x86 and either have a default or a static assert

Copy link
Collaborator

Choose a reason for hiding this comment

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

Probably should have it, but not here. And not in this PR.

However, I also wonder if there is a different option we should do. Basically so you don't have to do foolish things like only depend on symgc if you are on 64-bit systems.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think actually there will likely be a lot of compiler failures because symgc does a lot of asserts to ensure it can store everything properly.

@schveiguy
Copy link
Collaborator

I'm thinking we should add ldc/arm here as a compiler for CI. But I can't add that to master. So I'll probably just merge this and then add in a separate PR.

@schveiguy schveiguy merged commit e948528 into symmetryinvestments:master Aug 19, 2025
4 checks passed
@deadalnix
Copy link

There should be a C header somewhere in druntime defining the value of the futex syscall, no?

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.

4 participants