-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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
[RISCV][llvm-mca] Add llvm-mca tests for SiFive7 Vector Integer Arith… #65283
[RISCV][llvm-mca] Add llvm-mca tests for SiFive7 Vector Integer Arith… #65283
Conversation
The tests are failed. |
e0ebd04
to
837f1cf
Compare
I have updated the CHECK statements in the test case. I had run the update_mca_test_checks script before rebasing which led to the failure. |
048b0b9
to
c312c7e
Compare
…metic The intention of this test file long term is to test all valid (LMUL, SEW) pairs for each SchedWrite class. For this reason, we do not test every single instruction under every (LMUL, SEW) pair, since multiple instructions may use the same SchedWrite. For example, vadd.vv and vsub.vv both use the WriteVIALUV class. I didn't end up getting all (LMUL, SEW) pair for each SchedWrite class though. For example, vadd.vv and vadd.vx use WriteVIALUV and WriteVIALUX respectivley, but I treated all Vector Single-Width Integer Add and Subtract instructions as having the same behavior. I plan on improving the coverage as time goes on and figured this would be a good start. If there is any class of vector integer arithmetic instructions you'd like to see full coverage for in this patch, please let me know.
1721ea1
to
5d47ae4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
(maybe we should develop a test generator just like what we have in https://github.com/riscv-non-isa/rvv-intrinsic-doc/tree/main/rvv-intrinsic-generator?)
A test generator would be nice, although I think there are significantly less combinations compared to intrinsics. |
llvm#65283) …metic The intention of this test file long term is to test all valid (LMUL, SEW) pairs for each SchedWrite class. For this reason, we do not test every single instruction under every (LMUL, SEW) pair, since multiple instructions may use the same SchedWrite. For example, vadd.vv and vsub.vv both use the WriteVIALUV class. I didn't end up getting all (LMUL, SEW) pair for each SchedWrite class though. For example, vadd.vv and vadd.vx use WriteVIALUV and WriteVIALUX respectivley, but I treated all Vector Single-Width Integer Add and Subtract instructions as having the same behavior. I plan on improving the coverage as time goes on and figured this would be a good start. If there is any class of vector integer arithmetic instructions you'd like to see full coverage for in this patch, please let me know.
…metic
The intention of this test file long term is to test all valid (LMUL, SEW) pairs for each SchedWrite class. For this reason, we do not test every single instruction under every (LMUL, SEW) pair, since multiple instructions may use the same SchedWrite. For example, vadd.vv and vsub.vv both use the WriteVIALUV class.
I didn't end up getting all (LMUL, SEW) pair for each SchedWrite class though. For example, vadd.vv and vadd.vx use WriteVIALUV and WriteVIALUX respectivley, but I treated all Vector Single-Width Integer Add and Subtract instructions as having the same behavior. I plan on improving the coverage as time goes on and figured this would be a good start. If there is any class of vector integer arithmetic instructions you'd like to see full coverage for in this patch, please let me know.