ci: test on clang/arm64 #975
Conversation
|
I can't immediately tell why things are failing. Maybe try |
https://github.com/uber/h3/actions/runs/13526744257/job/37799141287#step:8:570 Every single test fails immediately. We should see if we can get it to dump the logs on failure, but I suspect it is accidentally trying to run x86-64 binaries in an arm64 environment. |
Only one check fails, and the rest in that group are cancelled early |
Not what I meant. If you open the link, you will see that every test is executed for that run and they all fail. That implies that it is not generating binaries that work for the platform. |
|
I'll enable the print-output-on-failure option in CMake so we can find out. I wonder if some of these sanitizers are not implemented/available on arm64. edit: dart-lang/sdk#55303 Seems to suggest we need a new version of Clang in order to use msan. |
|
Definitely better logging here. https://github.com/uber/h3/actions/runs/13529596356/job/37808380267#step:8:10522 Do these compile arg recommendations make sense for this test? |
|
@dfellis, ah, we were talking about different things. I mentioned That still might be helpful here, even if we only leave it on temporarily for this PR. |
|
From what I can tell, edit: OK, I'm just catching up: Maybe a newer version of clang would allow this, like you mentioned above, @isaacbrodsky |
|
So maybe drop this test for now to get things passing? And we can create a follow up issue to update clang and add msan. |
* ci: test on clang/arm64 * CTEST_OUTPUT_ON_FAILURE * test fail-fast * fix * exclude msan * simplify
No description provided.