[PATCH] D121370: [clang-format] SortIncludes should support "@import" lines in Objective-C

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 3 12:43:54 PDT 2022


krasimir added a comment.

In D121370#3488401 <https://reviews.llvm.org/D121370#3488401>, @kwk wrote:

> @krasimir could you please test this patch on your side?

@kwk thank you! the new version of the patch looks good!

I didn't know before about the issue you found with the `/* a comment before */ #include <foo>` (and since it's preexisting, if it's tricky to address, it would be reasonable to just file a bug for it and omit fixing it by this work, I think).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121370/new/

https://reviews.llvm.org/D121370



More information about the cfe-commits mailing list