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.

Sign up for GitHub

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

Jump to bottom

OS X 32 bit: real field alignment wrong, causes incorrect struct initialization #363

Closed
dnadlinger opened this issue May 20, 2013 · 2 comments
Closed

OS X 32 bit: real field alignment wrong, causes incorrect struct initialization #363

dnadlinger opened this issue May 20, 2013 · 2 comments
Labels
C-upstream-pending

Comments

@dnadlinger
Copy link
Member

The std.variant unit tests currently fail on 32 bit OS X. More specifically, the issue is that the testing struct in https://github.com/ldc-developers/phobos/blob/3d7211fedf3aff666d80e45e322e7cb63c87159b/std/variant.d#L1094 contains a real field, which is not properly initialized.

The reason for this is that the field offset info provided by the frontend is wrong, leading to incorrect codegen for the padding zeroing process: http://d.puremagic.com/issues/show_bug.cgi?id=10071
redstar pushed a commit that referenced this issue Sep 27, 2014
Merge pull request #363 from 9rnsr/fix9052
6055cfb 
Issue 9052 - [2.061 alpha] AA.length in a const context tries to call opAssign + no error line number
@kinke
Copy link
Member

kinke commented Nov 4, 2016

The front-end has been fixed in 2014, so we should be fine by now.

@kinke kinke closed this as completed May 12, 2018
@kinke
Copy link
Member

kinke commented May 12, 2018

32-bit macOS unittests are run since multilib-OSX-testing with CircleCI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-upstream-pending
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dnadlinger @kinke