[PATCH] D80270: [mlir][spirv] Adapt subview legalization to the updated op semantics.

[Mahesh Ravishankar via Phabricator via llvm-commits]

mravishankar created this revision.
Herald added subscribers: llvm-commits, jurahul, Kayjukh, frgossen, grosul1, bader, Joonsoo, stephenneuendorffer, liufengdb, lucyrfox, mgester, arpith-jacob, csigg, nicolasvasilache, antiagainst, shauheen, jpienaar, rriddle, mehdi_amini.
Herald added a reviewer: antiagainst.
Herald added a reviewer: herhut.
Herald added a project: LLVM.
mravishankar added a reviewer: hanchung.

The subview semantics changes recently to allow for more natural
representation of constant offsets and strides. The legalization of
subview op for lowering to SPIR-V needs to account for this.
Also change the linearization to use the strides from the affine map
of a memref.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D80270

Files:
  mlir/lib/Conversion/StandardToSPIRV/LegalizeStandardForSPIRV.cpp
  mlir/test/Conversion/GPUToSPIRV/load-store.mlir
  mlir/test/Conversion/GPUToSPIRV/loop.mlir
  mlir/test/Conversion/StandardToSPIRV/std-ops-to-spirv.mlir
  mlir/test/Conversion/StandardToSPIRV/subview-to-spirv.mlir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80270.265140.patch
Type: text/x-patch
Size: 18204 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200520/afea5668/attachment-0001.bin>