[all-commits] [llvm/llvm-project] 04481f: [mlir] Require std.alloc() ops to have canonical l...

[Christian Sigg via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 04481f26faff79027192bd27200956be16c11102
      https://github.com/llvm/llvm-project/commit/04481f26faff79027192bd27200956be16c11102
  Author: Christian Sigg <csigg at google.com>
  Date:   2020-11-16 (Mon, 16 Nov 2020)

  Changed paths:
    M mlir/lib/Conversion/StandardToLLVM/StandardToLLVM.cpp
    M mlir/test/Conversion/StandardToLLVM/standard-to-llvm.mlir

  Log Message:
  -----------
  [mlir] Require std.alloc() ops to have canonical layout during LLVM lowering.

The current code allows strided layouts, but the number of elements allocated is ambiguous. It could be either the number of elements in the shape (the current implementation), or the amount of elements required to not index out-of-bounds with the given maps (which would require evaluating the layout map).

If we require the canonical layouts, the two will be the same.

Reviewed By: nicolasvasilache, ftynse

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