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

Create bugs folder to outline found compiler bugs #127

Open
Quaker762 opened this issue Feb 14, 2018 · 4 comments
Open

Create bugs folder to outline found compiler bugs #127

Quaker762 opened this issue Feb 14, 2018 · 4 comments

Comments

@Quaker762
Copy link
Member

This is more for my own sanity and the sanity of others. I've hit 3-4 bugs in GCC (though I've only outlined 2) that people may encounter.

@z33ky You can see what I mean here: https://github.com/Quaker762/sh3redux/tree/bugs/bugs

@z33ky
Copy link
Collaborator

z33ky commented Feb 14, 2018

Regarding __attribute__((packed)), have you tried -mno-ms-bitfields?
I think the template-namespace bug is https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56480, correct? Please link to the upstream issue.

@Quaker762
Copy link
Member Author

Regarding attribute((packed)), have you tried -mno-ms-bitfields?

Ahh, I forgot about that switch. That seems to fix it on Windows. There are some reports however that it's still a bug in GCC, so it might vary per flavour of Linux?? This is kind of an odd one (see here: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54589; I've added this link to the top of the .cpp file)

I think the template-namespace bug is https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56480, correct? Please link to the upstream issue.

Yep, added it :)

@z33ky
Copy link
Collaborator

z33ky commented Feb 15, 2018

There are some reports however that it's still a bug in GCC, so it might vary per flavour of Linux?? This is kind of an odd one (see here: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54589; ...)

Did you link the right bug? That one seems to just be a missed optimization opportunity which only activates if the struct has a certain layout - and there's no mention of #pragma pack.

@Quaker762
Copy link
Member Author

Whoops, must've linked the wrong one. I'll get on that ASAP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants