Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add warning regarding different struct layouts on Apple M1 #211

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MoritzKn
Copy link

As discussed in #210

@c0gent
Copy link
Member

c0gent commented Sep 15, 2022

Let's see what you determine is the best solution for your issue based on my latest comment in #210 then we can update this with some documentation for future users encountering the same problem.

@MoritzKn
Copy link
Author

My understanding was that repr “C would create C ABI compatible structs (and other data types). Hence I interpreted the comment/documentation on OclPrm as allowing structs to be OclPrm as long as they have repr “C.
If that's not the case I'd simply clarify that statement. What data types does the statement still apply to? Single fields structs? Unions?

@c0gent
Copy link
Member

c0gent commented Nov 6, 2022

Ok I'll leave this open until I can come up with more clarifying documentation at some point.

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