compiler-explorer/compiler-explorer

[REQUEST]: Enable clang-tidy for Clang-CUDA builds

Open

#6,944 opened on Oct 8, 2024

View on GitHub
 (1 comment) (0 reactions) (0 assignees)TypeScript (14,391 stars) (1,605 forks)batch import
good first issuelang-cudarequest

Description

Is your feature request related to a problem? Please describe

No problem.

Describe the solution you'd like

Since Clang can be used to compile CUDA code, so can clang-tidy be used to lint it. Currently Compiler Explorer displays the tool is not supported, but technically it's possible. It should be fairly easy, pretty much drop-in replace "clang" with "clang-tidy" in the compiler invocation.

Describe alternatives you've considered

No alternatives.

Additional context

Not applicable

Contributor guide

Tech stack
cppc
Domain
toolingdeveloper experience
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
C++ basicsCUDA awarenessclang tidy
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75
Research direction
Look at how Compiler Explorer invokes clang for CUDA compilation (likely in lib/compilers/clang.js or similar). The change involves adding a new compiler configuration for clang tidy that mirrors the CUDA clang but uses 'clang tidy' as the executable. Check existing clang tidy support for non CUDA code to see the pattern. The issue notes it should be a drop in replacement, so ensure that the same flags and options are passed.