[PATCH] D79890: [MLIR] Continue renaming of "SideEffects"

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 27 21:16:12 PDT 2020


mehdi_amini added a comment.

In D79890#2058211 <https://reviews.llvm.org/D79890#2058211>, @rriddle wrote:

> I would have really preferred that we went in the opposite direction.


+1, I mentioned it on a previous revision: duplicating "Interface" in the path isn't useful.
I understood it would be coming later but first this was just aligning it all. @stephenneuendorffer can you clarify?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79890





More information about the llvm-commits mailing list