[all-commits] [llvm/llvm-project] d86047: [mlir][ArmSME] Update tile slice layout syntax (#6...

Cullen Rhodes via All-commits all-commits at lists.llvm.org
Mon Oct 16 02:55:43 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d86047cb665ecdb37d17fc83bae2f67d3a6455c4
      https://github.com/llvm/llvm-project/commit/d86047cb665ecdb37d17fc83bae2f67d3a6455c4
  Author: Cullen Rhodes <cullen.rhodes at arm.com>
  Date:   2023-10-16 (Mon, 16 Oct 2023)

  Changed paths:
    M mlir/include/mlir/Dialect/ArmSME/IR/ArmSMEOps.td
    M mlir/lib/Conversion/ArmSMEToSCF/ArmSMEToSCF.cpp
    M mlir/lib/Conversion/VectorToArmSME/VectorToArmSME.cpp
    M mlir/test/Conversion/ArmSMEToSCF/arm-sme-to-scf.mlir
    M mlir/test/Dialect/ArmSME/arm-sme-to-llvm.mlir
    M mlir/test/Dialect/ArmSME/roundtrip.mlir
    M mlir/test/Dialect/ArmSME/vector-ops-to-sme.mlir
    M mlir/test/Integration/Dialect/Vector/CPU/ArmSME/test-load-vertical.mlir

  Log Message:
  -----------
  [mlir][ArmSME] Update tile slice layout syntax (#69151)

This patch prefixes tile slice layout with `layout` in the
assemblyFormat:

  - `<vertical>`   -> `layout<vertical>`
  - `<horizontal>` -> `layout<horizontal>`

The reason for this change is the current format doesn't play nicely
with additional optional operands, required to support padding and
masking (#69148), as it becomes ambiguous.

This affects the the following ops:

  - arm_sme.tile_load
  - arm_sme.tile_store
  - arm_sme.load_tile_slice
  - arm_sme.store_tile_slice




More information about the All-commits mailing list