[PATCH] D129735: [WIP][RISCV] Add new pass to transform undef to pesudo for vector values.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 20 23:06:15 PDT 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVRVVInitUndef.cpp:167
+  switch (RegClassID) {
+  case RISCV::VRRegClassID:
+  case RISCV::VRNoV0RegClassID:
----------------
This is not maintainable.

I meant something like

```
const TargetRegisterClass *RC = MRI->getRegClass(Reg);
if (RC->hasSuperClassEq(&RISCV::VRRegClass)) {
  Opcode = RISCV::PseudoRVVInitUndefM1;
} else if (RC->hasSuperClassEq(&RRISCV::VRM2RegClass) {
  Opcode = RISCV::PseudoRVVInitUndefM2;
}
...
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129735/new/

https://reviews.llvm.org/D129735



More information about the llvm-commits mailing list