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? | |
3228 | Should it have HasStdExtV predicate? |
llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td | ||
---|---|---|
3228 | 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?