Skip to content
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.

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

fix(csr): CSRR instruction read xireg inOrder #4393

Merged
merged 1 commit into from
Mar 12, 2025
Merged

fix(csr): CSRR instruction read xireg inOrder #4393

merged 1 commit into from
Mar 12, 2025

Conversation

sinceforYy
Copy link
Contributor

@sinceforYy sinceforYy commented Mar 11, 2025

  • AIA registers are designed to be access asynchronously, so newCSR will wait for response. Therefore, CSRR instruction read mireg/sireg/vsireg inOrder.

@sinceforYy sinceforYy force-pushed the fix-csrr branch 2 times, most recently from 05bb9bd to 2fdfd33 Compare March 11, 2025 06:42
@XiangShanRobot
Copy link

[Generated by IPC robot]
commit: 2fdfd33

commit astar copy_and_run coremark gcc gromacs hmmer-Vector lbm linux mcf microbench milc namd povray wrf xalancbmk
2fdfd33 1.806 0.442 2.641 1.226 2.150 1.676 2.149 2.364 0.933 1.389 2.010 3.108 2.538 2.274 3.309

master branch:

commit astar copy_and_run coremark gcc gromacs hmmer-Vector lbm linux mcf microbench milc namd povray wrf xalancbmk
42b75a5 1.806 0.442 2.641 1.229 2.150 1.676 2.149 2.364 0.933 1.378 2.010 3.108 2.538 2.274 3.309
f9395f7 1.806 0.442 2.641 1.229 2.150 1.676 2.149 2.364 0.933 1.389 2.010 3.108 2.538 2.274 3.309
c1eb288 1.806 0.442 2.641 1.237 2.150 1.676 2.149 2.364 0.933 1.389 2.010 3.108 2.538 2.274 3.309
9d7a35d 1.822 0.442 2.641 1.228 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300
fad7c42 1.823 0.442 2.641 1.228 2.158 1.676 2.144 2.364 0.940 1.389 1.972 3.112 2.538 2.272 3.300
dfb03ba 1.822 0.442 2.641 1.234 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300
d6b0a27 1.823 0.442 2.641 1.226 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300
591ae1c 1.822 0.442 2.641 1.236 2.158 1.676 2.144 2.364 0.940 1.389 1.972 3.112 2.538 2.272 3.300
11269ca 1.823 0.442 2.641 1.231 2.158 1.676 2.144 2.364 0.940 1.389 1.972 3.112 2.538 2.272 3.300
7fbc1cb 1.823 0.442 2.641 1.233 2.158 1.676 2.144 2.364 0.940 1.378 1.972 3.112 2.538 2.272 3.300

@sinceforYy sinceforYy requested a review from Tang-Haojin March 12, 2025 01:43
@Tang-Haojin Tang-Haojin merged commit 8893eb2 into master Mar 12, 2025
10 checks passed
@Tang-Haojin Tang-Haojin deleted the fix-csrr branch March 12, 2025 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants