-
Notifications
You must be signed in to change notification settings - Fork 298
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
RVV
target test failures
#2078
Comments
Thanks for letting us know. Unfortunately RVV tests in our CI are currently disabled because the toolchain is crashing. We've filed an LLVM bug:) |
Meanwhile, LLVM has rolled back the patch that caused the compiler crash. Our RVV tests are again usable. Would you like to file an LLVM issue for the test failure on clang-19? |
Hi @jan-wassenberg , I have just rerun the tests with clang-17.0.6 and a recent version of clang (2ace7bd), and they both failed the The |
Interesting, thanks for checking. Our toolchain doesn't come with an exact version, but it is relatively close to LLVM HEAD. It seems to succeed with -march=rv64gcv1p0. What build flags are you using? |
I am using the default flags, and I believe that it is However, I found that the test only fails when VLEN is 128. What VLEN are you using for your CI? |
Oh, good catch! That's likely it. We seem to have 512-bit VLEN. Note that SortTag uses LMUL=1/2. The problem is that the base case is meant to handle at least two vectors (and Partition relies upon that), but the base case is also VL-dependent and we only handle up to 16 elements. Hopefully we can cap the vector size. |
The sort itself does check for the problem, but TestAllPartition did not, and soon will. |
PiperOrigin-RevId: 638680940
PiperOrigin-RevId: 638680940
Thanks. I tried the fa61572 commit, and it seems to work under VLEN=128/1024 on qemu. |
Thanks for confirming! |
PiperOrigin-RevId: 638680940
Hello, I compiled the HEAD version of highway (commit: 3cb5c1a) with a recent snapshot of clang-19 (commit:
44af53b
), and I got failed tests.The tests were run on qemu-riscv, and the following is the test outputs:
The text was updated successfully, but these errors were encountered: