-
Notifications
You must be signed in to change notification settings - Fork 322
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
HwyBlockwiseShiftTest.TestAllShiftRightLanes
test failing on Graviton3
#1938
Comments
Hi @bedroge , I think this might be caused by compiler gremlins, or perhaps a bug that I do not yet see, not the CPU: it is the expected value that is incorrect here. As far as I can see, the test and op are unchanged between 1.0.3 and the current code. Are you able to test with another compiler, in particular clang? |
I now tried it with GCC 12.3.0 as well, but seeing the same issue. Also tried GCC 13.2.0, but then the compilation fails with:
Not sure if I can easily try building it with Clang with EasyBuild, but I might give it a try. But just to be sure: you said that it's the expected value that's incorrect; so that means I can safely ignore this failing test? |
@jan-wassenberg Can you clarify how "serious" this failing test is? It is currently blocking us from including Highway into EESSI, and we're wondering if it really should, or whether we can ignore the failing test for now. So basically: is it a problem with the test, or is it really a signal of a problem with the installation of Highway? |
Hi @bedroge , sorry to hear the issue still affects GCC 13. Would you like to file a bug with the GCC bugzilla?
Yes, that's right. Though it does seem to point to a compiler bug which would be good for them to fix. Hi @boegel , I think it is reasonable to ignore this test especially if you are not using the ShiftRightLanes operation it is testing. The test does not signal a problem with the installation. The specific way it is failing shows that the compiler seems to have a bug affecting this test, which is unfortunately reasonably common. |
Thanks @jan-wassenberg. I've tried building it with Clang, and in that case everything did indeed work fine. But for now we will just ignore this failing test, as we would like to stick to GCC. |
Thanks for confirming clang works. Makes sense :) |
Would appreciate it if someone can help report this to GCC so they can fix it :) |
I'm trying to compile Highway 1.0.3 with EasyBuild using the following recipe: https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/h/Highway/Highway-1.0.3-GCCcore-12.2.0.eb.
The build completes without any issues, but one test is failing:
This is on a
c7g.4xlarge
instance in AWS, and this is thelscpu
output:Any idea why this test is failing?
The text was updated successfully, but these errors were encountered: