cpu: refactor ci, use local dramsim3 all - #590
Conversation
Change-Id: I3584235bf1dfbffd0844ba351197aab8247b90af
There was a problem hiding this comment.
Pull Request Overview
This PR refactors duplicate DRAMSim3 build steps across multiple GitHub Actions workflows into a reusable composite action to improve maintainability and consistency.
- Extracts repeated DRAMSim3 build logic into a centralized composite action at
.github/actions/build-dramsim/action.yml - Replaces 9 instances of inline DRAMSim3 build steps across 4 workflow files with calls to the new action
- Standardizes the build process to use local cache (
/nfs/home/share/gem5_ci/DRAMsim3) to avoid network issues
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.github/actions/build-dramsim/action.yml |
New composite action that builds DRAMSim3 from local cache |
.github/workflows/pr-quick-check.yml |
Replaced inline DRAMSim build with action reference |
.github/workflows/gem5.yml |
Replaced 6 inline DRAMSim build steps with action reference |
.github/workflows/gem5-vector.yml |
Replaced inline DRAMSim build with action reference |
.github/workflows/gem5-perf-template.yml |
Replaced inline DRAMSim build with action reference |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/run-spec |
🚀 Coremark Smoke Test Results
✅ Difftest smoke test passed! |
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
🚀 Coremark Smoke Test Results
✅ Difftest smoke test passed! |
7c4c716 to
2c5a23c
Compare
also add manually trigger CI perf tests Change-Id: Iccc2ee49ce632ed201be645f982a2f465b550b88
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
🚀 Coremark Smoke Test Results
✅ Difftest smoke test passed! |
Change-Id: I3584235bf1dfbffd0844ba351197aab8247b90af