[PATCH] D76258: NFC - Rename VectorOps Dialect to the Vector Dialect

Rob Suderman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 16 18:03:25 PDT 2020


rsuderman added a comment.

In D76258#1925536 <https://reviews.llvm.org/D76258#1925536>, @aartbik wrote:

> 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?


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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76258





More information about the llvm-commits mailing list