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(vnclip): use uimm instead of imm for vnclip_wi instructions #3894

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

Ziyue-Zhang
Copy link
Contributor

@Ziyue-Zhang Ziyue-Zhang commented Nov 19, 2024

image

Although the vs2 used by the vnclip instruction is signed, the immediate number is unsigned

@XiangShanRobot
Copy link

[Generated by IPC robot]
commit: 6bfa430

commit astar copy_and_run coremark gcc gromacs lbm linux mcf microbench milc namd povray wrf xalancbmk
6bfa430 1.917 0.450 2.701 1.230 2.840 2.464 2.398 0.921 1.430 2.069 3.437 2.716 2.387 3.261

master branch:

commit astar copy_and_run coremark gcc gromacs lbm linux mcf microbench milc namd povray wrf xalancbmk
aecf601
03426fe
39d5540
92f3664
3e2285d 1.917 0.450 2.701 1.226 2.840 2.464 2.398 0.921 1.430 2.069 3.437 2.716 2.387 3.261

@Tang-Haojin Tang-Haojin added the cherry-pick needed This pull request needs to be cherry-picked to master label Nov 21, 2024
@Tang-Haojin Tang-Haojin merged commit 6266af8 into master Nov 21, 2024
9 checks passed
@Tang-Haojin Tang-Haojin deleted the fix-vnclip branch November 21, 2024 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherry-pick needed This pull request needs to be cherry-picked to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants