-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[C++][Compute] _mm256_set_m128i()
is unavailable on OpenSUSE 15.1 (GCC 7.5)
#44098
Comments
By some googling I can find people seeing similar issue, such as [1]. The root cause seems to point to a GCC bug [2], which is claimed to be fixed in 8.0. I think I do have some workaround in mind. But I want to check first if we are able to upgrade GCC to 8.0 for this particular CI job? Thanks. [1] https://gitlab.com/libeigen/eigen/-/issues/2045 |
@assignUser Is R 4.1 still maintained? (Can we remove |
@kou yes we support the last 4 versions so 4.1 to 4.4 |
OK. @zanmato1984 Could you try a workaround for the platform? |
Though I am not sure if the opensuse job is really important, it's not an official flavour and tbh I don't think there is a huge R demographic on it... @jonkeane what do you think? |
I filed PR #44116 to workaround it anyway. |
I see we already have a PR for that, which is good, but for some added context on why that's good
It would be good to maintain the matrix of support if we can, I don't know of any other exception to our matrix (i.e. "this works on these versions of R and these OSes, except this version of R + OS combo"), so adding one shouldn't be taken lightly. Additionally + possibly more importantly opensuse is one of the Posit Public Package Manager platforms, so they build there. |
…missing it (#44116) ### Rationale for this change AVX2 intrinsic _mm256_set_m128i is missing in GCC versions <8.0 - this is a GCC bug discussed in [1], causing certain CI build failed. [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80582 ### What changes are included in this PR? Check the GCC version and use a home made replacement if necessary. ### Are these changes tested? Manually tested. ### Are there any user-facing changes? None. * GitHub Issue: #44098 Authored-by: Ruoxi Sun <zanmato1984@gmail.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
Issue resolved by pull request 44116 |
Describe the bug, including details regarding any error messages, version, and platform.
test-r-rstudio-r-base-4.1-opensuse155
is failing with the following error:https://dev.azure.com/ursacomputing/crossbow/_build/results?buildId=66380&view=logs&j=0da5d1d9-276d-5173-c4c4-9d4d4ed14fdb&t=6c939d89-0d1a-51f2-8b30-091a7a82e98c&l=1511
GH-43389 may be related.
Component(s)
C++
The text was updated successfully, but these errors were encountered: