Renaming the VectorOps dialect to Vector to better match the names of the other Dialects. This is part of a larger rename to change FooOps dialects as the Ops suffix is redundant.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Time | Test | |
---|---|---|
1,280 ms | LLVM.CodeGen/AArch64::Unknown Unit Message ("") |
Event Timeline
Comment Actions
Although purely mechanical, this is of course a very visible change. Can you state a bit more in the description on where, when, and why the rename was decided?
Comment Actions
Updated the description. Overall the rename is for consistency and to avoid redundancy. All dialects have Ops so including the "Ops" suffix is unneeded. LoopOps and AffineOps are undergoing similar renames, so updating the VectorOps dialect to match.
mlir/include/mlir/Dialect/Vector/VectorOps.h | ||
---|---|---|
14 | The second one should still be VectorOps to match the file name. |
The second one should still be VectorOps to match the file name.