[all-commits] [llvm/llvm-project] af9f7d: NFC. Clean up memref utils library

Uday Bondhugula via All-commits all-commits at lists.llvm.org
Wed Mar 9 02:32:04 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: af9f7d319b8c12063a9d105c9e13984141a51c62
      https://github.com/llvm/llvm-project/commit/af9f7d319b8c12063a9d105c9e13984141a51c62
  Author: Uday Bondhugula <uday at polymagelabs.com>
  Date:   2022-03-09 (Wed, 09 Mar 2022)

  Changed paths:
    M mlir/include/mlir/Dialect/MemRef/IR/MemRef.h
    M mlir/include/mlir/Dialect/MemRef/Utils/MemRefUtils.h
    M mlir/lib/Dialect/Bufferization/IR/BufferizationOps.cpp
    M mlir/lib/Dialect/Bufferization/Transforms/BufferUtils.cpp
    M mlir/lib/Dialect/MemRef/IR/CMakeLists.txt
    M mlir/lib/Dialect/MemRef/IR/MemRefDialect.cpp
    M mlir/lib/Dialect/MemRef/Utils/CMakeLists.txt
    M mlir/lib/Dialect/MemRef/Utils/MemRefUtils.cpp

  Log Message:
  -----------
  NFC. Clean up memref utils library

NFC. Clean up memref utils library. This library had a single function
that was completely misplaced. MemRefUtils is expected to be (also per
its comment) a library providing analysis/transforms utilities on memref
dialect ops or memref types. However, in reality it had a helper that
was depended upon by the MemRef dialect, i.e., it was a helper for the
dialect ops library and couldn't contain anything that itself depends on
the MemRef dialect. Move the single method to the memref dialect that
will now allow actual utilities depending on the memref dialect to be
placed in it.

Put findDealloc in the `memref` namespace. This is a pure move.

Differential Revision: https://reviews.llvm.org/D121273




More information about the All-commits mailing list