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

Fixed calling_conventions tests on MinGW #257

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

Conversation

Kojoley
Copy link
Contributor

@Kojoley Kojoley commented Mar 27, 2019

On MinGW calling conversion are actually macros that are expanded into a GCC attribute (e.g. __fastcall expands into __attribute__((__fastcall__))).

On MinGW calling conversion are actually macros that are expanded into a GCC
attribute (e.g. `__fastcall` expands into `__attribute__((__fastcall__))`).
@Kojoley
Copy link
Contributor Author

Kojoley commented Apr 2, 2019

Ping

@stefanseefeld
Copy link
Member

Sorry for my silence. Me not being a Windows developer I feel a bit uncomfortable reviewing this. Do you think it would be possible to add a MinGW (cross-)compiler to the mix of compilers we use on Travis-CI or AppVeyor ? That would help prove that things work as intended.

@Kojoley
Copy link
Contributor Author

Kojoley commented Apr 2, 2019

I seen other boost projects use mingw on appveyor, I could have added it here, but I do know nothing about faber (and it seems to not run module tests #254).

@stefanseefeld
Copy link
Member

OK, fair point. I'll thus try to add MinGW to the mix, then we should be able to see whether this PR fixes one (or more) failures.

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