diff options
| author | Lorenzo Chelini <l.chelini@icloud.com> | 2023-04-26 11:08:31 +0200 |
|---|---|---|
| committer | Lorenzo Chelini <l.chelini@icloud.com> | 2023-04-27 10:09:45 +0200 |
| commit | 61be9358963806f6dfd118cd7602a67e8ec01c82 (patch) | |
| tree | 3a74727cefbd4d454f6d01be81d4d2045cc7dee3 /llvm/lib/CodeGen/MachineModuleInfoImpls.cpp | |
| parent | 1e63f8272e8ad2b41f9c4d58250d7de264c4aa4b (diff) | |
| download | llvm-61be9358963806f6dfd118cd7602a67e8ec01c82.zip llvm-61be9358963806f6dfd118cd7602a67e8ec01c82.tar.gz llvm-61be9358963806f6dfd118cd7602a67e8ec01c82.tar.bz2 | |
[MLIR][Linalg] Change destination logic in `bubbleUpPackOpThroughGenericOp`.
In `bubbleUpPackOpThroughGenericOp`, we replaced the init operands with
a new `tensor.empty` if the operation was a pure element-wise op. This
behaviour is not wrong but not ideal because we "break" the original
use-def-chain of the output operand by materializing a new
`tensor.empty`. We should use `tensor.empty` as a destination *only* if the
initial init operand was already a `tensor.empty`, as we do in
`PushDownUnpack`.
Reviewed By: hanchung
Differential Revision: https://reviews.llvm.org/D149250
Diffstat (limited to 'llvm/lib/CodeGen/MachineModuleInfoImpls.cpp')
0 files changed, 0 insertions, 0 deletions