commit | 681f54a46ef879ba448a86a6e81c0200b6571126 | [log] [tgz] |
---|---|---|
author | Stephen Neuendorffer <stephen.neuendorffer@xilinx.com> | Tue Nov 30 14:37:30 2021 -0800 |
committer | Copybara-Service <copybara-worker@google.com> | Tue Nov 30 15:21:13 2021 -0800 |
tree | 0b2f205b9f13ad423563e20a197bc127860431c7 | |
parent | b9580d8325d061a1d18cb32bef26328c11785aad [diff] |
Revert "[MLIR] Update Vector To LLVM conversion to be aware of assume_alignment" This reverts commit 29a50c5864ddab283c1ff38694fb5926ce37b39a. After LLVM lowering, the original patch incorrectly moved alignment information across an unconstrained GEP operation. This is only correct for some index offsets in the GEP. It seems that the best approach is, in fact, to rely on LLVM to propagate information from the llvm.assume() to users. Thanks to Thomas Raoux for catching this. GitOrigin-RevId: 73863648892ee7063c7fd4e658d7614fd721504a
See https://mlir.llvm.org/ for more information.