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?