Skip to content

Add separate GMMA microbenchmark build target#79

Merged
JRPan merged 1 commit intodevfrom
dev-jr
Feb 7, 2026
Merged

Add separate GMMA microbenchmark build target#79
JRPan merged 1 commit intodevfrom
dev-jr

Conversation

@JRPan
Copy link
Contributor

@JRPan JRPan commented Feb 7, 2026

Summary

  • Filter GMMA directories out of the default GPU_Microbenchmark build so they don't break standard compilation
  • Add dedicated gmma / GPU_Microbenchmark_gmma Make targets to compile GMMA benchmarks independently
  • Add corresponding clean_gmma target for cleanup

Test plan

  • Run make GPU_Microbenchmark and verify GMMA directories are excluded
  • Run make GPU_Microbenchmark_gmma and verify only GMMA benchmarks are built
  • Verify make clean in GPU_Microbenchmark still works for non-GMMA targets

🤖 Generated with Claude Code

Filter GMMA directories out of the default microbenchmark build and
add dedicated `gmma` / `GPU_Microbenchmark_gmma` targets so GMMA
benchmarks can be compiled independently without affecting the
standard build.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Contributor Author

@JRPan JRPan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review

Changes look good. Here's a summary of what this PR does:

src/cuda/GPU_Microbenchmark/Makefile

  • Renames SUB_DIRS to ALL_SUB_DIRS to hold all subdirectories, then splits them:
    • GMMA_DIRS — directories ending in gmma/ (via $(filter %gmma/))
    • SUB_DIRS — everything else (via $(filter-out))
  • Adds gmma and clean_gmma targets that mirror the existing all / clean targets but operate only on GMMA directories.

src/Makefile

  • Adds a top-level GPU_Microbenchmark_gmma target that invokes the new gmma sub-target and moves the resulting binaries into the output directory.

Notes

  • The %gmma/ filter pattern will match any subdirectory whose name ends with gmma. If a non-GMMA benchmark name ever ends in gmma it would be incorrectly filtered — unlikely but worth keeping in mind.
  • The existing all and clean targets are unaffected for non-GMMA benchmarks; this is a purely additive change.

LGTM ✅

@JRPan JRPan merged commit 69888de into dev Feb 7, 2026
2 checks passed
@JRPan JRPan deleted the dev-jr branch February 10, 2026 23:53
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.

1 participant