Skip to content

Remove nvcc arch flags from clangd#9996

Open
Jacobfaib wants to merge 1 commit into
NVIDIA:mainfrom
Jacobfaib:jacobf/2026-07-17/clangd-fixes
Open

Remove nvcc arch flags from clangd#9996
Jacobfaib wants to merge 1 commit into
NVIDIA:mainfrom
Jacobfaib:jacobf/2026-07-17/clangd-fixes

Conversation

@Jacobfaib

@Jacobfaib Jacobfaib commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Description

clang does not understand -arch=native and thinks it is -march instead

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@Jacobfaib Jacobfaib self-assigned this Jul 17, 2026
@Jacobfaib
Jacobfaib requested a review from a team as a code owner July 17, 2026 14:41
@Jacobfaib
Jacobfaib requested a review from jrhemstad July 17, 2026 14:41
@github-project-automation github-project-automation Bot moved this to Todo in CCCL Jul 17, 2026
@cccl-authenticator-app cccl-authenticator-app Bot moved this from Todo to In Review in CCCL Jul 17, 2026
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: f3a4b43f-13d5-4bcc-991c-f54b1c7d0de9

📥 Commits

Reviewing files that changed from the base of the PR and between e7c9b65 and c31fdb9.

📒 Files selected for processing (1)
  • .clangd

📝 Walkthrough

Summary by CodeRabbit

  • Bug Fixes
    • Improved CUDA tooling compatibility by correctly filtering GPU architecture compiler arguments.

Walkthrough

Changes

CUDA clangd filtering

Layer / File(s) Summary
Update CUDA architecture flag filtering
.clangd
The CompileFlags filter now strips --gpu-architecture* arguments instead of -arch* arguments.

Comment @coderabbitai help to get the list of available commands.

@Jacobfaib Jacobfaib changed the title Remove nvcc arch flags from clang args Remove nvcc arch flags from clangd Jul 17, 2026
@Jacobfaib
Jacobfaib enabled auto-merge (squash) July 17, 2026 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

3 participants