Febbe added a comment.

I think I am done, and you can review it now.

With the latest changes, SortPriority is no longer required to match Priority, 
it can always be 0, since Priority is now used in the sorting algorithm.
This is what the documentation tends to depict:

"and #includes are sorted first according to increasing category number and 
then alphabetically within each category."
" There is a third and optional field SortPriority which can used while 
IncludeBlocks = IBS_Regroup to define the priority in which #includes should be 
ordered."


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143691

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to