[all-commits] [llvm/llvm-project] 0d9247: [mlir] [VectorOps] Merge VectorReduction/VectorRed...

[Aart Bik via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 0d924700a610c59555ede59325d3c51515679903
      https://github.com/llvm/llvm-project/commit/0d924700a610c59555ede59325d3c51515679903
  Author: aartbik <ajcbik at google.com>
  Date:   2020-03-05 (Thu, 05 Mar 2020)

  Changed paths:
    M mlir/include/mlir/Dialect/VectorOps/VectorOps.td
    M mlir/lib/Conversion/VectorToLLVM/ConvertVectorToLLVM.cpp
    M mlir/lib/Dialect/VectorOps/VectorOps.cpp
    M mlir/lib/Dialect/VectorOps/VectorTransforms.cpp
    M mlir/test/Dialect/VectorOps/invalid.mlir
    M mlir/test/Dialect/VectorOps/ops.mlir
    M mlir/test/Dialect/VectorOps/vector-contract-transforms.mlir

  Log Message:
  -----------
  [mlir] [VectorOps] Merge VectorReduction/VectorReductionV2 into one Op

Summary:
Paying off some technical debt in VectorOps, where I introduced a special
op for a fused accumulator into reduction to avoid some issues around
printing and parsing an optional accumulator. This CL merges the two
into one op again and does things the right way (still would be nice
to have "assemblyFormat" for optional operands though....).

Reviewers: nicolasvasilache, andydavis1, ftynse, rriddle

Reviewed By: nicolasvasilache

Subscribers: mehdi_amini, rriddle, jpienaar, burmako, shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester, lucyrfox, liufengdb, Joonsoo, llvm-commits

Tags: #llvm

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