EEW=64 of mulh and its vairants requires V extension.
Authored by: Craig Topper <craig.topper@sifive.com> @craig.topper
Paths
| Differential D117947
[RISCV] Don't allow i64 vector div by constant to use mulh with Zve64x ClosedPublic Authored by eopXD on Jan 22 2022, 12:47 AM.
Details Summary EEW=64 of mulh and its vairants requires V extension. Authored by: Craig Topper <craig.topper@sifive.com> @craig.topper
Diff Detail
Event TimelineHerald added subscribers: VincentWu, luke957, achieveartificialintelligence and 24 others. · View Herald TranscriptJan 22 2022, 12:47 AM
Comment Actions Concluding the test case change:
eopXD marked 3 inline comments as done. Comment ActionsUndo comment moved and follow clang-format.
This revision is now accepted and ready to land.Jan 24 2022, 10:05 AM This revision was landed with ongoing or failed builds.Jan 25 2022, 9:55 AM Closed by commit rGb089e4072a01: [RISCV] Don't allow i64 vector div by constant to use mulh with Zve64x (authored by eopXD). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 402566 llvm/lib/Target/RISCV/RISCVISelLowering.cpp
llvm/lib/Target/RISCV/RISCVSubtarget.h
llvm/test/CodeGen/RISCV/rvv/vdiv-sdnode.ll
llvm/test/CodeGen/RISCV/rvv/vdivu-sdnode.ll
llvm/test/CodeGen/RISCV/rvv/vrem-sdnode.ll
llvm/test/CodeGen/RISCV/rvv/vremu-sdnode.ll
|
Do we need to move this comment down? It still applies at its previous location, doesn't it?