Enhance and clean up xrt_test and shim_test interfaces#1049
Conversation
There was a problem hiding this comment.
Pull request overview
This PR enhances and cleans up the xrt_test and shim_test interfaces by restructuring test data paths, removing deprecated functionality, and improving test execution patterns. The changes streamline the testing framework by consolidating path handling, removing unused parameters, and enhancing help output.
Changes:
- Reorganized test data paths to use architecture-specific and test-specific subdirectories
- Removed deprecated
-sand-dcommand-line options and related functionality - Refactored stress test implementations to use consistent execution patterns with timing metrics
- Enhanced help messages to include available test cases listing
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| test/xrt_test/xrt_test.cpp | Restructured test data paths, removed deprecated options, refactored stress tests, moved runlist test, and enhanced help output |
| test/shim_test/shim_test.cpp | Added test listing to help output |
| test/shim_test/dev_info.cpp | Updated workspace paths to new directory structure |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
hlaccabu
force-pushed
the
xrt_test
branch
2 times, most recently
from
d1aabc8 to
b764baf
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
hlaccabu
force-pushed
the
xrt_test
branch
2 times, most recently
from
1c5c2de to
59d41de
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
hlaccabu
force-pushed
the
xrt_test
branch
2 times, most recently
from
4e147d0 to
96ab106
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Hayden Laccabue <hlaccabu@amd.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2 participants
No description provided.