Reduce code duplication: Move various helper functions, that are duplicated in TensorDialect, MemRefDialect, LinalgDialect, StandardDialect, into a new StaticValueUtils.cpp.
Depends On D104676
Paths
| Differential D104687
[mlir][NFC] Cleanup: Move helper functions to StaticValueUtils ClosedPublic Authored by springerm on Jun 21 2021, 10:21 PM.
Details Summary Reduce code duplication: Move various helper functions, that are duplicated in TensorDialect, MemRefDialect, LinalgDialect, StandardDialect, into a new StaticValueUtils.cpp. Depends On D104676
Diff Detail
Event TimelineHerald added subscribers: dcaballe, cota, mravishankar and 18 others. · View Herald TranscriptJun 21 2021, 10:21 PM rriddle added inline comments.
This revision now requires changes to proceed.Jun 21 2021, 10:33 PM
Comment Actions LGTM, thanks!
This revision is now accepted and ready to land.Jun 24 2021, 11:21 AM This revision was landed with ongoing or failed builds.Jun 26 2021, 11:57 PM Closed by commit rG0813700de1af: [mlir][NFC] Cleanup: Move helper functions to StaticValueUtils (authored by springerm). · Explain Why This revision was automatically updated to reflect the committed changes. springerm marked 5 inline comments as done.
Revision Contents
Diff 353827 mlir/include/mlir/Dialect/Linalg/IR/LinalgOps.td
mlir/include/mlir/Dialect/Linalg/Transforms/Transforms.h
mlir/include/mlir/Dialect/StandardOps/IR/Ops.h
mlir/include/mlir/Dialect/Utils/StaticValueUtils.h
mlir/include/mlir/Interfaces/ViewLikeInterface.h
mlir/include/mlir/Interfaces/ViewLikeInterface.td
mlir/lib/Conversion/StandardToLLVM/StandardToLLVM.cpp
mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
mlir/lib/Dialect/Linalg/Transforms/ComprehensiveBufferize.cpp
mlir/lib/Dialect/Linalg/Transforms/Vectorization.cpp
mlir/lib/Dialect/MemRef/IR/CMakeLists.txt
mlir/lib/Dialect/MemRef/IR/MemRefOps.cpp
mlir/lib/Dialect/StandardOps/IR/Ops.cpp
mlir/lib/Dialect/Tensor/IR/CMakeLists.txt
mlir/lib/Dialect/Tensor/IR/TensorOps.cpp
mlir/lib/Dialect/Utils/CMakeLists.txt
mlir/lib/Dialect/Utils/StaticValueUtils.cpp
|
Is the mlir:: necessary on these?