[mlir][SparseTensor][ArmSVE] Fix missing lli substitutions
authorCullen Rhodes <cullen.rhodes@arm.com>
Tue, 2 May 2023 14:30:02 +0000 (14:30 +0000)
committerCullen Rhodes <cullen.rhodes@arm.com>
Tue, 2 May 2023 14:43:48 +0000 (14:43 +0000)
commit707b6e94b88a5bac96157d03756f359060458d08
tree4ae1ffada2b54bcf6b46bf7adde1014d50c52d30
parent2353b52eae71ffa3993d21f0bed0c571cf2e0c79
[mlir][SparseTensor][ArmSVE] Fix missing lli substitutions

The MLIR SVE integration tests are now enabled in the
clang-aarch64-full-2stage buildbot under emulation (QEMU) and two of the
sparse integration tests are failing [1]:

  * mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_sorted_coo.mlir
  * mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_spmm.mlir

The reason for this is the SVE RUN lines use plain 'lli' rather than the
'%lli_host_or_aarch64_cmd' substitution that's necessary to run under
emulation. The CI doesn't support SVE so the tests will SIGILL unless
run under emulation.

I should note the logs don't show a SIGILL, only the non-descript:

  FileCheck error: '<stdin>' is empty.

but I expect this is what's actually happening.

https://lab.llvm.org/buildbot/#/builders/179/builds/6051/steps/12/logs/stdio
mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_sorted_coo.mlir
mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_spmm.mlir