[PATCH] D115810: [RISCV] Don't allow vector types to be used with inline asm 'r' constraint
Jessica Clarke via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 15 11:01:30 PST 2021
jrtc27 added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:9589
case 'r':
- return std::make_pair(0U, &RISCV::GPRRegClass);
+ if (!VT.isVector())
+ return std::make_pair(0U, &RISCV::GPRRegClass);
----------------
```
if (VT.isVector())
break;
```
is a bit more extensible in that line lengths don't get stupidly long as the list grows; we exclude our own register class from r downstream like that.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115810/new/
https://reviews.llvm.org/D115810
More information about the llvm-commits
mailing list