Add vamoswap, vamoadd, vamoxor, vamoand, vamoor,
vamomin, vamomax, vamominu, vamomaxu intrinsics.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| llvm/include/llvm/IR/IntrinsicsRISCV.td | ||
|---|---|---|
| 589 | based on https://reviews.llvm.org/D93359#inline-872142 | |
| llvm/include/llvm/IR/IntrinsicsRISCV.td | ||
|---|---|---|
| 589 | Good idea, thanks! | |
Nitpick: this would be better titled "[RISCV] Add intrinsics for vector AMO instructions" - I was a little confused seeing the title come past :)
| llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td | ||
|---|---|---|
| 107 | vlxe and vlse have the same logic and I think they can reuse above utility functions, right? | |
| 3216 | Should it have HasStdExtV predicate? | |
| llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td | ||
|---|---|---|
| 3216 | I think FeatureExtZvamo implies HasStdExtV in RISCV.td | |
| llvm/include/llvm/IR/IntrinsicsRISCV.td | ||
|---|---|---|
| 476 | Is the value the same type as the result? Can we use LLVMMatchType<0> for it? | |
- fix value type to match result type
- share octuple utility function with vlxei & vsxei patterns
Is the value the same type as the result? Can we use LLVMMatchType<0> for it?