Skip to content

[cppinterop] Delete each test's interpreters when it ends - #24035

Open
aaronj0 wants to merge 1 commit into
root-project:masterfrom
aaronj0:cppinterop-test-interp-cleanup
Open

aaronj0 wants to merge 1 commit into
root-project:masterfrom
aaronj0:cppinterop-test-interp-cleanup

Conversation

@aaronj0

@aaronj0 aaronj0 commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

@bellenot can you test with this patch?

CppInterOpTests kept every interpreter alive for the whole run, about
250 by the end. On a 32-bit Windows build the newest of them sit near
the 2 GB address-space limit when the run crashes in a late test.

Three tests relied on an interpreter left behind by an earlier test and
now create their own.
@bellenot

bellenot commented Nov 6, 2026

Copy link
Copy Markdown
Member
1: [  SKIPPED ] CUDATest.CUB_BlockReduceWithDeviceFunction
1: [  FAILED  ] 1 test, listed below:
1: [  FAILED  ] CppInterOpTest/InProcessJIT.FunctionReflection_JitCallAdvanced, where TypeParam = InProcessJITConfig
1:
1:  1 FAILED TEST
1:   YOU HAVE 1 DISABLED TEST
1:
1/1 Test #1: cppinterop-CppInterOpTests .......***Failed   26.54 sec

@github-actions

github-actions Bot commented Nov 7, 2026

Copy link
Copy Markdown

Test Results

    21 files      21 suites   2d 22h 28m 34s ⏱️
 3 862 tests  3 860 ✅ 0 💤  2 ❌
65 095 runs  65 074 ✅ 0 💤 21 ❌

For more details on these failures, see this check.

Results for commit b2ca4ae.

This branch has not been deployed

No deployments
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.

3 participants