[clang] clang-format: Add IncludeSortKey option (PR #137840)

Björn Schäpers via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 26 00:58:38 PDT 2025


================
@@ -1647,7 +1647,7 @@ FormatStyle getLLVMStyle(FormatStyle::LanguageKind Language) {
   LLVMStyle.SeparateDefinitionBlocks = FormatStyle::SDS_Leave;
   LLVMStyle.ShortNamespaceLines = 1;
   LLVMStyle.SkipMacroDefinitionBody = false;
-  LLVMStyle.SortIncludes = {/*Enabled=*/true, /*IgnoreCase=*/false};
+  LLVMStyle.SortIncludes = {/*Enabled=*/true, /*IgnoreCase=*/false, /*IgnoreExtension=*/false};
----------------
HazardyKnusperkeks wrote:

Currently it wouldn't change anything until the option is enabled. For me this change is good, I did just not vote, because of @mydeveloperday request for changes.

And as cleared, the the "wrong" sorting was due using `A.cpp` as filename and thus `A.h` becomes the main header and is put up front.

https://github.com/llvm/llvm-project/pull/137840


More information about the cfe-commits mailing list