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(redirect): fix the blockCommit condition #3992

Merged
merged 1 commit into from
Dec 7, 2024
Merged

Conversation

Anzooooo
Copy link
Member

@Anzooooo Anzooooo commented Dec 5, 2024

The frontend needs to send redirect before sending commit.
When flushPipe occurs continuously, it may not meet the above condition in some cases, so this logic is added.

@XiangShanRobot
Copy link

[Generated by IPC robot]
commit: a73e677

commit astar copy_and_run coremark gcc gromacs lbm linux mcf microbench milc namd povray wrf xalancbmk
a73e677 1.959 0.450 2.686 1.237 2.832 2.461 2.393 0.919 1.407 1.992 3.435 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
2bff79a 2.687 0.919 1.407 3.261
02d2ded 1.959 0.450 2.686 1.228 2.832 2.461 2.393 0.919 1.407 1.992 3.435 2.709 2.383 3.261
4aa305e 1.959 0.450 2.687 1.232 2.832 2.461 2.393 0.919 1.407 1.992 3.434 2.709 2.383 3.261
189833a 1.959 0.450 2.686 1.231 2.832 2.461 2.393 0.919 1.407 1.992 3.434 2.709 2.383 3.261
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

@Tang-Haojin Tang-Haojin merged commit 35477a7 into master Dec 7, 2024
9 checks passed
@Tang-Haojin Tang-Haojin deleted the wzz-fix-block branch December 7, 2024 01:59
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.

3 participants