This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse][gpu] enable sm80+ sparsity integration test only when explicitly set
ClosedPublic

Authored by K-Wu on Jun 14 2023, 3:02 PM.

Diff Detail

Event Timeline

K-Wu created this revision.Jun 14 2023, 3:02 PM
Herald added a project: Restricted Project. · View Herald Transcript
K-Wu requested review of this revision.Jun 14 2023, 3:02 PM
K-Wu updated this revision to Diff 531530.Jun 14 2023, 3:14 PM

more specific dir name

aartbik added inline comments.Jun 14 2023, 3:52 PM
mlir/test/CMakeLists.txt
41

I think we should use something like

if (MLIR_INCLUDE_INTEGRATION_TESTS)

if (MLIR_ENABLE_CUDA_CUSPARSE)
    set(MLIR_RUN_CUDA_SM80_TESTS)
     if(MLIR_ENABLE_CUDA_CUSPARSELT)
          set(MLIR_RUN_CUDA_SM80_LT_TESTS ON)

i.e. the INTEGRATION flag and the two CUDA flags automatically set the SM80

aartbik added inline comments.Jun 14 2023, 3:56 PM
mlir/test/CMakeLists.txt
41

probably document our decision
So something like

  1. When the Integration tests are requested via the MLIR_INCLUDE_INTEGRATION_TESTS configuration flag
  2. we automatically include sm80 tests when build for cuSparse when the configuration flag MLIR_ENABLE_CUDA_CUSPARSE
  3. is set and include sm80 lt tests when the MLIR_ENABLE_CUDA_CUSPARSELT is set in addition to those.
K-Wu updated this revision to Diff 531566.Jun 14 2023, 4:23 PM

add files

K-Wu updated this revision to Diff 531568.Jun 14 2023, 4:30 PM

incorporating comments and documentation

K-Wu marked 2 inline comments as done.Jun 14 2023, 4:30 PM
aartbik added inline comments.Jun 14 2023, 4:37 PM
mlir/test/CMakeLists.txt
46

you are missing the outermost

if (MLIR_INCLUDE_INTEGRATION_TESTS)
K-Wu added inline comments.Jun 14 2023, 5:29 PM
mlir/test/CMakeLists.txt
46

Actually, L16 does this guard

aartbik accepted this revision.Jun 15 2023, 9:28 AM
aartbik added inline comments.
mlir/test/CMakeLists.txt
46

Ah, apologies, I did not scroll up enough ;-)

This revision is now accepted and ready to land.Jun 15 2023, 9:28 AM
K-Wu marked 2 inline comments as done.Jun 15 2023, 9:56 AM
K-Wu added inline comments.
mlir/test/CMakeLists.txt
46

No problem! Thanks for keeping a sharp eye!

This revision was landed with ongoing or failed builds.Jun 15 2023, 10:44 AM
This revision was automatically updated to reflect the committed changes.
K-Wu marked an inline comment as done.