This is an archive of the discontinued LLVM Phabricator instance.

[mlir][aarch64] Enable MLIR integration tests for SVE/SME under emulation
ClosedPublic

Authored by c-rhodes on Apr 13 2023, 7:26 AM.

Details

Summary

This enables MLIR integrations tests for SVE and SME to run under
emulation with QEMU, as discussed on D148111.

Diff Detail

Event Timeline

c-rhodes created this revision.Apr 13 2023, 7:26 AM
c-rhodes requested review of this revision.Apr 13 2023, 7:26 AM
DavidSpickett accepted this revision.Apr 13 2023, 7:34 AM

So we'll have the SVE bots running native SVE tests, and this one running SVE and SME emulated. Sounds good to me.

@awarzynski any thoughts?

This revision is now accepted and ready to land.Apr 13 2023, 7:34 AM
Matt added a subscriber: Matt.Apr 13 2023, 7:38 AM
awarzynski accepted this revision.Apr 13 2023, 8:11 AM

Makes sense, thanks! LGTM

Thanks for the quick review all. This depends on D148111, I'll land once that's merged.

This has now landed, @gkistanova please could you restart the bot?

I noticed buildbot has been restarted, there's a bunch of tests failing at the moment, the HWASAN are unrelated to this, but this patch has exposed issues in the following tests that are failing:

MLIR :: Integration/Dialect/SparseTensor/CPU/concatenate_dim_1.mlir
MLIR :: Integration/Dialect/SparseTensor/CPU/sparse_sorted_coo.mlir
MLIR :: Integration/Dialect/SparseTensor/CPU/sparse_spmm.mlir

The problem common to all of them is they don't use emulation since they're not using substitutions. The top one is now using mlir-cpu-runner since a recent change in D146917, but %lli should have been replaced with an equivalent substitution to support emulation. @awarzynski is looking into this. The bottom two are more straightforward, they simply weren't using a substitution to begin with. I landed 707b6e94b88a that should hopefully fix them.

I noticed buildbot has been restarted, there's a bunch of tests failing at the moment, the HWASAN are unrelated to this, but this patch has exposed issues in the following tests that are failing:

MLIR :: Integration/Dialect/SparseTensor/CPU/concatenate_dim_1.mlir
MLIR :: Integration/Dialect/SparseTensor/CPU/sparse_sorted_coo.mlir
MLIR :: Integration/Dialect/SparseTensor/CPU/sparse_spmm.mlir

The problem common to all of them is they don't use emulation since they're not using substitutions. The top one is now using mlir-cpu-runner since a recent change in D146917, but %lli should have been replaced with an equivalent substitution to support emulation. @awarzynski is looking into this. The bottom two are more straightforward, they simply weren't using a substitution to begin with. I landed 707b6e94b88a that should hopefully fix them.

Thanks @c-rhodes ! I've updated concatenate_dim_1.mlir here: https://github.com/llvm/llvm-project/commit/20bf8c403c5a8db6107358e534fb3ab5dd0ae9a3. https://reviews.llvm.org/D148005 will bring a proper fix (just need to resurrect it first)