[all-commits] [llvm/llvm-project] ff52ad: [mlir] Change DenseArrayAttr to TensorType

Jeff Niu via All-commits all-commits at lists.llvm.org
Mon Aug 1 19:17:43 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ff52ad796c971dbf805375a2140344f742db94a3
      https://github.com/llvm/llvm-project/commit/ff52ad796c971dbf805375a2140344f742db94a3
  Author: Jeff Niu <jeff at modular.com>
  Date:   2022-08-01 (Mon, 01 Aug 2022)

  Changed paths:
    M mlir/lib/IR/AsmPrinter.cpp
    M mlir/lib/IR/BuiltinAttributes.cpp

  Log Message:
  -----------
  [mlir] Change DenseArrayAttr to TensorType

Previously, DenseArrayAttr used VectorType for its shaped type.
VectorType is problematic for arrays because it doesn't support zero
dimensions, meaning that an empty array would have `vector<i32>` as its
type. ElementsAttr would think that an empty dense array is size 1, not
0. This patch switches over to TensorType, which does support zero
dimensions.

Fixes #56860

Reviewed By: mehdi_amini

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




More information about the All-commits mailing list