Skip to content

Conversation

@guitargeek
Copy link
Contributor

@guitargeek guitargeek commented Nov 25, 2025

This is to test the newer GCC 14 on ARM.

Should also add coverage for #14446.

Two commits with fixes to h2root and RVec tests are also added to avoid these test failures on alma10 ARM without any further changes:

TEST FAILURES:
317:gtest-math-vecops-vecops-rvec
1998:roottest-root-hist-h2root

@guitargeek guitargeek requested a review from dpiparo as a code owner November 25, 2025 00:39
@guitargeek guitargeek self-assigned this Nov 25, 2025
@github-actions
Copy link

github-actions bot commented Nov 25, 2025

Test Results

    21 files      21 suites   3d 22h 8m 30s ⏱️
 3 778 tests  3 777 ✅ 0 💤 1 ❌
77 472 runs  77 471 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 16c3d8b.

♻️ This comment has been updated with latest results.

@guitargeek guitargeek force-pushed the alma10_arm branch 3 times, most recently from a4682f5 to b92c0fa Compare November 26, 2025 11:44
@guitargeek guitargeek requested a review from pcanal as a code owner November 26, 2025 12:28
is_special: true
property: march_native
overrides: ["CMAKE_BUILD_TYPE=RelWithDebInfo", "CMAKE_CXX_FLAGS=-march=native", "CMAKE_C_FLAGS=-march=native", "builtin_zlib=ON", "builtin_zstd=ON"]
- image: alma9
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do the experiment use for arm64? i.e. shouldn't we still test both alma9 and alma10.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know, but I would prefer to not blow up the CI matrix too much for no reason. If you want, we can also keep Alma 9. In the end, the motivation for this PR was to demo that aarch64 works with gcc 14, which is shipped with Alma 10. If I can show that, #14446 can be closed.

In any case, if the fixes to h2root and the RVec tests successfully fix the problems with Alma 10 ARM, I will open independent PRs for these fixes.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dpiparo What coverage do we need for arm64/alma?

This is to test the newer C++ compilers on ARM.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants