kadircet added a comment.
i thought clang-format was also putting angled includes and "OtherHeaders" into
same category, by looking at:
LLVMStyle.IncludeStyle.IncludeCategories = {
{"^\"(llvm|llvm-c|clang|clang-c)/", 2, 0, false},
{"^(<|\"(gtest|gmock|isl|json)/)", 3, 0, false}, // Starts with
`<` or gtest/gmock...
{".*", 1, 0, false}};
not sure why it is done so (stylistic preferences i suppose ...). it definitely
makes sense for me to separate them into 2 different categories, but i wouldn't
dare make that decision :D maybe ask on cfe-dev first to get some consensus.
but it would make sense to include isl and json in this list for now, i forgot
to do that in the initial patch :(
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93354/new/
https://reviews.llvm.org/D93354
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
