[PATCH] D93446: [RISCV] Add vadd with mask and without mask builtin.

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 18 12:36:03 PST 2020


craig.topper added inline comments.


================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:17569
+      // The order of intrinsic operands is (maskedoff, op1, op2, mask, vl).
+      SmallVector<llvm::Value*, 4> ShuffleOps(Ops.size());
+      ShuffleOps[0] = Ops[1];
----------------
I think this whole sequence is equivalent to

```
std::rotate(Ops.begin(), Ops.begin() + 1, Ops.end() - 1);
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93446



More information about the cfe-commits mailing list