Skip to content

Avx #163

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

Merged
merged 5 commits into from
Nov 1, 2017
Merged

Avx #163

merged 5 commits into from
Nov 1, 2017

Conversation

gwenn
Copy link
Contributor

@gwenn gwenn commented Nov 1, 2017

No description provided.

gwenn added 5 commits November 1, 2017 10:13
8 levels of macro expansion takes too long to compile.
Macro expansion can be reduced to four levels
Copy/paste from avx2::_mm256_blend_epi32
@alexcrichton alexcrichton merged commit 46d64f0 into rust-lang:master Nov 1, 2017
@alexcrichton
Copy link
Member

👍 Thanks!

@gwenn gwenn deleted the avx branch November 1, 2017 14:48
@gnzlbg
Copy link
Contributor

gnzlbg commented Nov 1, 2017

@alexcrichton this PR broke the rustfmt build. We should either check that it doesn't break or... just allow the build to fail because of it. I mean, worst case we can't commit one day because rustfmt fails to build but... that is fixed quickly, and as soon as it is shipped with rustup this will never fail.

@alexcrichton
Copy link
Member

@gnzlbg oh I'm in "if it's green we merge it" mode, I'm not looking at the test results to see if the "allow fail" entries are failing.

@gnzlbg
Copy link
Contributor

gnzlbg commented Nov 2, 2017 via email

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.

3 participants