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.

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

feat(ECC): add ECC for coupledL2 and bump coupledL2 & openLLC #3808

Merged
merged 5 commits into from
Dec 5, 2024
Merged

feat(ECC): add ECC for coupledL2 and bump coupledL2 & openLLC #3808

merged 5 commits into from
Dec 5, 2024

Conversation

Ma-YX
Copy link
Contributor

@Ma-YX Ma-YX commented Oct 29, 2024 •
edited by linjuanZ
Loading

  • add optional ECC for L2 Cache tag & data
  • add Poison & (optional) DataCheck for CHI

@Ma-YX
submodule(coupledL2): bump coupledL2
022a3ca 
enable DataCheck option
@Ma-YX Ma-YX marked this pull request as ready for review December 5, 2024 01:30
@Ma-YX Ma-YX changed the title feat(ECC): add ECC for coupledL2 and bump coupledL2 feat(ECC): add ECC for coupledL2 and bump coupledL2 & openLLC Dec 5, 2024
@XiangShanRobot
Copy link

[Generated by IPC robot]
commit: 2c9c1c7

commit astar copy_and_run coremark gcc gromacs lbm linux mcf microbench milc namd povray wrf xalancbmk
2c9c1c7 1.959 0.450 2.686 1.235 2.832 2.461 2.393 0.919 1.407 1.992 3.434 2.709 2.383 3.261

master branch:

commit astar copy_and_run coremark gcc gromacs lbm linux mcf microbench milc namd povray wrf xalancbmk
189833a
e10e20c 1.959 0.450 2.687 1.228 2.832 2.461 2.393 0.919 1.407 1.992 3.435 2.709 2.383 3.261
9fabe32 1.959 0.450 2.687 1.227 2.832 2.461 2.393 0.919 1.407 1.992 3.435 2.709 2.383 3.261
8c9adf0 1.959 0.450 2.686 1.221 2.832 2.461 2.393 0.919 1.407 1.992 3.435 2.709 2.383 3.264
c92e74d

@linjuanZ linjuanZ merged commit 4aa305e into master Dec 5, 2024
9 checks passed
@linjuanZ linjuanZ deleted the l2ECC branch December 5, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@linjuanZ linjuanZ linjuanZ approved these changes

@Ivyfeather Ivyfeather Awaiting requested review from Ivyfeather Ivyfeather is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Ma-YX @XiangShanRobot @linjuanZ