Skip to content

Enhancement- Add SM80/90 architecture to CUTLASS builds for CUDA 12.8+ for A100 - #785

Open
Hongtao Zhang (polarG) wants to merge 3 commits into
mainfrom
fix/cutlass-add-sm80-arch
Open

Enhancement- Add SM80/90 architecture to CUTLASS builds for CUDA 12.8+ for A100#785
Hongtao Zhang (polarG) wants to merge 3 commits into
mainfrom
fix/cutlass-add-sm80-arch

Conversation

@polarG

Copy link
Copy Markdown
Contributor

Add sm_80 to the CUTLASS architecture lists to ensure Ampere GPU support is included in builds targeting CUDA 12.8+ for A100

Add sm_80 to the CUTLASS architecture lists to ensure Ampere GPU
support is included in builds targeting CUDA 12.8+ and 12.9+.
@polarG
Hongtao Zhang (polarG) requested a review from a team as a code ownerMarch 11, 2026 23:00
CopilotAI review requested due to automatic review settings March 11, 2026 23:00

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the CUTLASS build configuration to ensure Ampere (A100 / SM80) is included in CUTLASS builds when using CUDA 12.8+.

Changes:

  • Add sm_80 to the CUTLASS ARCHS list for CUDA >= 12.8.
  • Expand the CUDA >= 12.9 CUTLASS ARCHS list to include 80 and 90 in addition to 100/103.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadthird_party/Makefile Outdated
@codecov

codecovBot commented Mar 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.70%. Comparing base (6b8e810) to head (b8cfc30).

Additional details and impacted files
@@ Coverage Diff @@## main #785 +/- ##
=======================================
Coverage 85.70% 85.70% =======================================
Files 102 102 Lines 7703 7703 =======================================
Hits 6602 6602 Misses 1101 1101 
FlagCoverage Δ
cpu-python3.10-unit-test70.96% <ø> (ø)
cpu-python3.12-unit-test70.96% <ø> (ø)
cpu-python3.7-unit-test70.43% <ø> (ø)
cuda-unit-test83.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@polarGHongtao Zhang (polarG) changed the title Bugfix: Add SM80/90 architecture to CUTLASS builds for CUDA 12.8+ for A100Bugfix - Add SM80/90 architecture to CUTLASS builds for CUDA 12.8+ for A100Mar 14, 2026
Hongtao Zhangand others added 2 commits March 17, 2026 20:45
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
CopilotAI review requested due to automatic review settings March 17, 2026 20:46

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadthird_party/Makefile
Comment on lines 46 to 51
if [ -d cutlass ]; then rm -rf cutlass; fi
git clone --branch v4.1.0 --depth 1 https://github.com/NVIDIA/cutlass.git && cd cutlass
else ifeq ($(shell echo $(CUDA_VER)">=12.8" | bc -l), 1)
$(eval ARCHS := "90;100")
$(eval ARCHS := "80;90;100")
if [ -d cutlass ]; then rm -rf cutlass; fi
git clone --branch v3.9.2 --depth 1 https://github.com/NVIDIA/cutlass.git && cd cutlass

@guoshzhaoguoshzhaoApr 13, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hongtao Zhang (@polarG) Please address the comment from Copilot.

@polarG
Hongtao Zhang (polarG) enabled auto-merge (squash) March 20, 2026 17:53
@polarGHongtao Zhang (polarG) changed the title Bugfix - Add SM80/90 architecture to CUTLASS builds for CUDA 12.8+ for A100Enhancement- Add SM80/90 architecture to CUTLASS builds for CUDA 12.8+ for A100Mar 23, 2026
@polarGHongtao Zhang (polarG) added enhancement New feature or request labels Mar 23, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@polarG@guoshzhao