Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FORMAT] Add .clang-format for C++ code #1404

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

nileshnegi
Copy link
Collaborator

Details

Work item:
#1340

What were the changes?
Add a .clang-format file for formatting C++ code

Approval Checklist

Do not approve until these items are satisfied.

  • Verify the CHANGELOG has been updated, if
    • there are any NCCL API version changes,
    • any changes impact library users, and/or
    • any changes impact any other ROCm library.

.clang-format Outdated

AttributeMacros: ['__host__', '__device__', '__global__', '__forceinline__', '__shared__', '__launch_bounds__']
# Trick clang into thinking that our C-style attributes are C++-style attributes
Macros:
Copy link
Contributor

@mustafabar mustafabar Nov 27, 2024

Choose a reason for hiding this comment

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

this fails on Ubuntu clang-format version 14.0.0-1ubuntu1.1 (try one of the banff nodes). It complains about the Macros word usage

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