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

Update clang-format version to 18 #2945

Merged
merged 3 commits into from
Mar 22, 2025
Merged

Conversation

AmirhosseinPoolad
Copy link
Contributor

Ubuntu 24.04 installs clang-format version 18 by default and this PR updates the version used by CI and 'make format' from 14 to 18. Since clang-format 18 is a bit smarter than 14, there were some parts of the code where it fixed some formatting issues that version 14 did not find problematic. The second commit of this PR fixes those issues.

@github-actions github-actions bot added VPR VPR FPGA Placement & Routing Tool lang-cpp C/C++ code infra Project Infrastructure build Build system lang-shell Shell scripts (bash etc.) labels Mar 21, 2025
@AmirhosseinPoolad AmirhosseinPoolad force-pushed the fix_clang_format_version branch from 700800e to 62623d3 Compare March 21, 2025 00:52
@AmirhosseinPoolad AmirhosseinPoolad force-pushed the fix_clang_format_version branch from 62623d3 to 1a433b4 Compare March 21, 2025 01:02
This is the version that is installed by default on Ubuntu 24.04
which we currently run CI and testing on.
@AmirhosseinPoolad AmirhosseinPoolad force-pushed the fix_clang_format_version branch from 1a433b4 to 115c6d4 Compare March 21, 2025 20:12
@AmirhosseinPoolad AmirhosseinPoolad force-pushed the fix_clang_format_version branch from 115c6d4 to 1cecbab Compare March 21, 2025 20:51
Copy link
Contributor

@AlexandreSinger AlexandreSinger left a comment

Choose a reason for hiding this comment

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

LGTM

@AlexandreSinger AlexandreSinger merged commit e7f3a02 into master Mar 22, 2025
36 checks passed
@AlexandreSinger AlexandreSinger deleted the fix_clang_format_version branch March 22, 2025 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Build system infra Project Infrastructure lang-cpp C/C++ code lang-shell Shell scripts (bash etc.) libvtrutil VPR VPR FPGA Placement & Routing Tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants