[all-commits] [llvm/llvm-project] 488132: [mlir][vector][NFC] Document and simplify vector c...

[SeongJaePark via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 48813264376be9868bec78fa1653bc9003db512f
      https://github.com/llvm/llvm-project/commit/48813264376be9868bec78fa1653bc9003db512f
  Author: SeongJaePark <111725324+SeongjaeP at users.noreply.github.com>
  Date:   2026-06-11 (Thu, 11 Jun 2026)

  Changed paths:
    M mlir/lib/Dialect/Vector/Transforms/LowerVectorContract.cpp
    M mlir/test/Dialect/Vector/vector-contract-to-parallel-arith-transforms.mlir

  Log Message:
  -----------
  [mlir][vector][NFC] Document and simplify vector contract reshape helpers (#200544)

Replace the bare `// TODO` placeholders above `reshapeLoad` and
`reshapeStore`
in `LowerVectorContract.cpp` with doc comments describing each helper's
semantics (the `index == -1` / `index == 0` / `index > 0` cases) and
short
before/after IR examples.
  
Also drop the redundant `VectorType` parameter from both helpers; the
type is
now inferred from the value operand (`val` for `reshapeLoad`, `result`
for
`reshapeStore`).
  
Add a LIT test that exercises the recursive `reshapeLoad` unroll path by
mapping a parallel iterator to a non-leading dimension, which falls
through
to the generic `lowerParallel` / `reshapeLoad` path.

No functional change.

---------

Co-authored-by: sjae.park <dev at opt-ai.kr>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications