Skip to content

[Feature][Codegen] Remove the legacy Text SunMMIO backend path and Refocus SunMMIO skeleton tests on traversal coverage completeness#138

Merged
xiaoyao-NKU merged 3 commits intoSUNMMIO:dev/codegenfrom
firefrogliu666:feature/remove-sunmmio-text-builder
May 7, 2026
Merged

[Feature][Codegen] Remove the legacy Text SunMMIO backend path and Refocus SunMMIO skeleton tests on traversal coverage completeness#138
xiaoyao-NKU merged 3 commits intoSUNMMIO:dev/codegenfrom
firefrogliu666:feature/remove-sunmmio-text-builder

Conversation

@firefrogliu666
Copy link
Copy Markdown
Collaborator

PR Summary

This PR has two main goals:

  • Remove the legacy Text SunMMIO backend path
  • Refocus SunMMIO skeleton tests on traversal coverage completeness

1) Text backend removal (SUVM-only path)

  • Removes TextSunMMIOBuilder and its text-specific code path.
  • Removes backend selection plumbing (text vs suvm) from SunMMIO codegen/runtime integration.
  • Makes target.build.tilelang_sunmmio_without_compile use SUVM path only (with backend validation/default behavior aligned to SUVM).
  • Keeps builder abstraction in place (no broad architecture refactor).

2) Skeleton test refocus

  • Reworks testing/python/target/test_sunmmio_codegen_skeleton.py to make coverage report the primary oracle for codegen_sunmmio.cc traversal.
  • Keeps minimal smoke/failure tests (non-empty source, While/Shuffle unsupported, compile-path not implemented).
  • Removes brittle text-format assertions.
  • Keeps synthetic coverage kernels and adds a real TileLang frontend kernel in coverage parameterization.

Coverage Oracle

  • Uses:
    • TL_SUNMMIO_CODEGEN_COVERAGE_PATH
    • TL_SUNMMIO_CODEGEN_COVERAGE_STRICT=1
  • Asserts:
    • report exists
    • required keys exist
    • missing_node_types == []
    • missing_call_ops == []

Validation

  • Ran:
    • PYTHONPATH=/home/liuchunfeng/Tilelang-Mesh-Sync-Pc/python:/home/liuchunfeng/Tilelang-Mesh-Sync-Pc /home/liuchunfeng/.conda/envs/tl/bin/python -m pytest -q testing/python/target/test_sunmmio_codegen_skeleton.py
  • Result:
    • 8 passed

@firefrogliu666 firefrogliu666 reopened this May 7, 2026
@xiaoyao-NKU xiaoyao-NKU merged commit 6b066c7 into SUNMMIO:dev/codegen May 7, 2026
3 of 4 checks passed
@firefrogliu666 firefrogliu666 deleted the feature/remove-sunmmio-text-builder branch May 8, 2026 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants