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

Does this #13

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Does this #13

wants to merge 3 commits into from

Conversation

bmschmidt
Copy link
Owner

@bmschmidt bmschmidt commented Jun 17, 2016

I noticed a Stack Overflow user saying that this fork by fixes the Windows 8 installation bug. I was hoping @cpeeples might see this tag and be able to explain what the changes to Makevars.win are doing. Do they fix the versions of Windows that don't work? Is there any reason to think they might break the versions of Windows that are already working? Unfortunately I can't test on Windows myself.

@cpeeples
Copy link

I added the -mno-avx compiler flag after noticing some errors when trying to install in my Win7 environment. I don't know a lot about Advanced Vector Extensions, but I was able to track down the 'illegal instruction' errors well enough to find that they had something to do with AVX. Adding the -mno-avx flag allowed me to install the package. I think the worst downside of including this compiler flag might be sub-optimal performance.

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