Skip to content

[WIP] System Verilog Testing #3 #3114

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 28 commits into
base: master
Choose a base branch
from
Draft

[WIP] System Verilog Testing #3 #3114

wants to merge 28 commits into from

Conversation

loglav03
Copy link

@loglav03 loglav03 commented Jun 5, 2025

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed

AlexandreSinger and others added 22 commits February 7, 2025 17:44
@AlexandreSinger
[CI] Added System Verilog Regression Tests to GitHub Runners
f41ea8c 
The system verilog regression tests were put into the currently
not-working nightly tests. Since they are so small, putting them in the
GitHub runners tests instead so they do not regress.

The strong parmys test also has system verilog tests (I think); however,
that test may be too large for the GitHub runners. We may consider
moving that test to the SAVI workflows in the future.
@amirarjmand93
Update CMakeLists.txt
b2c2052
@amirarjmand93
Update CMakeLists.txt
5d8eb3a 
Change install path
@amirarjmand93
Update golden results and CMakeLists
4a48f33
@amirarjmand93
Update CMakeLists.txt
00595e2
@amirarjmand93
Update CMakeLists.txt
385b981
@amirarjmand93
Update CMakeLists.txt
dcc3258 
paths corrected
@amirarjmand93
Update test.yml
8a9125a 
check synlig path
@amirarjmand93
Update CMakeLists.txt
f45e2cd 
update CPU core
@amirarjmand93
Update CMakeLists.txt
ecd77cd 
Run the default cmake --build
@amirarjmand93
Update CMakeLists.txt
2033ae8 
reduce build proc
@amirarjmand93
Update CMakeLists.txt
0227c24 
Synlig Hardcoded Paths resolved
@amirarjmand93
Update paths.py
cabc382 
update synlig path
@amirarjmand93
Update CMakeLists.txt
72b816e
@amirarjmand93
Update test.yml
c3305f5 
edited to relaunch the test
@amirarjmand93
Update paths.py
965472e
@amirarjmand93
Update CMakeLists.txt
7e3089b
@amirarjmand93
Update CMakeLists.txt
14ab8a2 
change CMAKE_BINARY_DIR to CMAKE_CURRENT_SOURCE_DIR. CI use separate build directions.
@amirarjmand93
Update paths.py
6f43262
@amirarjmand93
Update CMakeLists.txt
5a43529 
add debug message
@amirarjmand93
Update CMakeLists.txt
3a46e76
@amirarjmand93
Update CMakeLists.txt
d1ae4ab 
limited cpu core
@github-actions github-actions bot added infra Project Infrastructure lang-python Python code lang-make CMake/Make code external_libs labels Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
external_libs infra Project Infrastructure lang-make CMake/Make code lang-python Python code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@loglav03 @AlexandreSinger @amirarjmand93