[Mlir-commits] [mlir] [mlir][memref] Rewrite scalar `memref.copy` through reinterpret_cast into load/store (PR #186118)

[Paul Kirth]

================
@@ -11,6 +11,11 @@
 
 include "mlir/Pass/PassBase.td"
 
+def BypassReinterpretCastPass : Pass<"memref-bypass-reinterpret-cast"> {
----------------
ilovepi wrote:

```suggestion
def SimplifyReinterpretCastPass : Pass<"memref-simplify-reinterpret-cast"> {
```
or maybe

```suggestion
def ElideReinterpretCastPass : Pass<"memref-elide-reinterpret-cast"> {
```
maybe these capture the intent better than `bypass`? this is a rather minor detail,  I was a bit surprised by the name. Up to you on changing it or not.

https://github.com/llvm/llvm-project/pull/186118