[PATCH] D152940: [RISCV] Make all vector binops use the _TU pseudo form
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 16 11:22:03 PDT 2023
craig.topper requested changes to this revision.
craig.topper added inline comments.
This revision now requires changes to proceed.
================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVSDPatterns.td:91
+ instruction_name#"_VV_"# vlmul.MX#"_TU"))
+ (op_type (IMPLICIT_DEF)),
op_reg_class:$rs1,
----------------
This should be `result_type`
================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVSDPatterns.td:115
+ instruction_name#_#suffix#_# vlmul.MX#"_TU"))
+ (vop_type (IMPLICIT_DEF)),
vop_reg_class:$rs1,
----------------
`result_type`
================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVSDPatterns.td:166
+ instruction_name#"_"#vlmul.MX#"_TU"))
+ (vop_type (IMPLICIT_DEF)),
vop_reg_class:$rs1,
----------------
result_type
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152940/new/
https://reviews.llvm.org/D152940
More information about the llvm-commits
mailing list