Fix managed memory smoke test#9654
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis change modifies a CUDA smoke test for cudaMallocManaged. Previously, the test would skip execution when device managed-memory support was absent; now it uses REQUIRE to assert support unconditionally, causing the test to fail instead of skip on unsupported devices. ChangesSmoke Test Assertion Update
important: This change causes the test to fail (rather than skip) on devices/platforms lacking managed memory support. Confirm all CI/runner targets actually support managed memory, or this will break builds on unsupported hardware. Comment |
Description
CCCL tests require managed memory. For instance, all thrust tests that are added with
DECLARE_VECTOR_UNITTESTcheck if universal vector works. Current smoke implementation didn't trigger an error if system hasn't supported managed memory.Checklist