Skip to content

Harden JIT 373472 regression test gating for 32-bit processes#129540

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-jit64-test-failure
Draft

Harden JIT 373472 regression test gating for 32-bit processes#129540
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-jit64-test-failure

Conversation

Copilot AI commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

This addresses a persistent x86 Checked failure in JIT/jit64/regress/vsw/373472/test_il/test_il.dll caused by large contiguous allocations that are unreliable on 32-bit address space. The test’s existing skip logic was too narrow and did not cover all affected 32-bit processes.

  • Skip condition alignment
    • Updated src/tests/JIT/jit64/regress/vsw/373472/test.cs to gate the test with PlatformDetection.Is32BitProcess (instead of IsArmProcess).
  • IL twin consistency
    • Updated src/tests/JIT/jit64/regress/vsw/373472/test.il to use the same Is32BitProcess gate and aligned the ActiveIssue message text with the C# version.
  • Net effect
    • Keeps intended coverage on unaffected platforms while avoiding known non-deterministic failures from 32-bit contiguous allocation limits.
[ActiveIssue(
    "Allocates large contiguous array that is not consistently available on 32-bit platforms",
    typeof(PlatformDetection),
    nameof(PlatformDetection.Is32BitProcess))]

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @akoeplinger, @matouskozak, @simonrozsival
See info in area-owners.md if you want to be subscribed.

Co-authored-by: JulieLeeMSFT <63486087+JulieLeeMSFT@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot June 17, 2026 22:27
Copilot AI changed the title [WIP] Fix test failure for JIT/jit64 regression test Harden JIT 373472 regression test gating for 32-bit processes Jun 17, 2026
Copilot AI requested a review from JulieLeeMSFT June 17, 2026 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants