[xharness] Clean up ARM64 test variations.#25736
Conversation
ARM64 is the default everywhere now, and it's the only architecture we'll run on anyways, so there's no need to condition anything on arm64 being available. This also makes it possible to remove some redundant test configurations. Additionally, make 'supports_interpreter' conditioned on 'supports_mono', since that's really what it is.
There was a problem hiding this comment.
Pull request overview
This PR simplifies xharness’ Jenkins test variation generation by treating ARM64 as the baseline architecture (removing ARM64 availability gating and redundant ARM64-only variants), and by aligning interpreter support with Mono support.
Changes:
- Remove ARM64-availability conditioning and ARM64-specific duplicate test variations.
- Always emit the simulator NativeAOT variations with an explicit simulator RID (arm64) where required.
- Define
supports_interpreterin terms ofsupports_mono(instead of duplicating platform logic).
✅ [PR Build #f2c6a53] Build passed (Detect API changes) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
✅ [PR Build #f2c6a53] Build passed (Build packages) ✅Pipeline on Agent |
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
✅ [PR Build #f2c6a53] Build passed (Build macOS tests) ✅Pipeline on Agent |
🚀 [CI Build #f2c6a53] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 207 tests passed 🎉 Tests counts✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Monterey (12): All 5 tests passed. Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
ARM64 is the default everywhere now, and it's the only architecture we'll run
on anyways, so there's no need to condition anything on arm64 being available.
This also makes it possible to remove some redundant test configurations.
Additionally, make 'supports_interpreter' conditioned on 'supports_mono',
since that's really what it is.