Skip to content
This repository was archived by the owner on Aug 30, 2024. It is now read-only.

WIP: remove -msse2 from linux32 builds, use explicit build tag #80

Merged
merged 2 commits into from
Apr 24, 2020

Conversation

mattip
Copy link
Contributor

@mattip mattip commented Apr 24, 2020

Checking if removing the -msse2 flag from 32-bit fixes wheel building. Should revert this once the last 1.18 build is done, since the problem is fixed on master and the flag allows faster loops.

@mattip
Copy link
Contributor Author

mattip commented Apr 24, 2020

Yup, now all builds are green, there are manylinux1 and manylinux2010 builds. We should consider reverting this when the 1.18 branch dies in order to boost performance on linux32.

The build from pushing the commit to end was 36 minutes. Since this branch is in my fork it did not upload, which might add a bit more time.

CFLAGS="-msse2 -std=c99 -fno-strict-aliasing"
# Causes failure for pre-1.19 in np.reciprocal
# CFLAGS="-msse2 -std=c99 -fno-strict-aliasing"
CFLAGS="-std=c99 -fno-strict-aliasing"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hope to get 1.18.4 out using the travis/appveyor system, so this is really an azure testing option. Any idea why the test failures don't show up with travis/appveyor? Once we have things working we can start pointing to numpy master branch for numpy-wheels master, sometimes we want to trigger a nightly build unrelated to cron.

@charris charris merged commit c80e842 into MacPython:master Apr 24, 2020
@charris
Copy link
Contributor

charris commented Apr 24, 2020

Thanks Matti.

@mattip
Copy link
Contributor Author

mattip commented Apr 24, 2020

The old build system does not use the -msse2 flag

@charris
Copy link
Contributor

charris commented Apr 24, 2020

The old build system does not use the -msse2 flag

Ah...

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

Successfully merging this pull request may close these issues.

2 participants