Skip to content

Upgrade CppInterOp to master#21961

Merged
aaronj0 merged 1 commit intoroot-project:masterfrom
aaronj0:cppinterop-upgrade
Apr 23, 2026
Merged

Upgrade CppInterOp to master#21961
aaronj0 merged 1 commit intoroot-project:masterfrom
aaronj0:cppinterop-upgrade

Conversation

@aaronj0
Copy link
Copy Markdown
Contributor

@aaronj0 aaronj0 commented Apr 20, 2026

Brings in fixes for GetType, support for tracing crashes, per-interpreter JitCall wrapper cache, improved handling of the interpreter creation and destruction

@aaronj0 aaronj0 self-assigned this Apr 20, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 20, 2026

Test Results

    22 files      22 suites   3d 8h 18m 10s ⏱️
 3 850 tests  3 848 ✅  1 💤 1 ❌
76 007 runs  75 988 ✅ 18 💤 1 ❌

For more details on these failures, see this check.

Results for commit aa70d49.

♻️ This comment has been updated with latest results.

@aaronj0 aaronj0 force-pushed the cppinterop-upgrade branch 7 times, most recently from ae13591 to fedb695 Compare April 23, 2026 08:38
@aaronj0 aaronj0 force-pushed the cppinterop-upgrade branch from fedb695 to aa70d49 Compare April 23, 2026 08:39
@aaronj0 aaronj0 requested a review from guitargeek April 23, 2026 11:22
@aaronj0
Copy link
Copy Markdown
Contributor Author

aaronj0 commented Apr 23, 2026

Test failure on alma10 looks unrelated. The windows CI seems to run out of space
cc @guitargeek @bellenot

Copy link
Copy Markdown
Contributor

@guitargeek guitargeek left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@aaronj0
Copy link
Copy Markdown
Contributor Author

aaronj0 commented Apr 23, 2026

The failing tests on alma10 is unrelated and visible on master: https://github.com/root-project/root/actions/runs/24824068443/job/72655489006

@aaronj0 aaronj0 merged commit c8626da into root-project:master Apr 23, 2026
70 of 77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants