[PATCH] D150083: [clang-format] ObjCPropertyAttributeOrder to sort ObjC property attributes

Owen Pan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 15 00:35:46 PST 2023


owenpan added a comment.

BTW, it may be simpler and more efficient to use a set (e.g. `llvm::SmallSet`) for `Indices`, especially if we don't need/want to handle duplicate attributes that have a value. (See D150083#inline-1551778 <https://reviews.llvm.org/D150083#inline-1551778>.)


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

https://reviews.llvm.org/D150083



More information about the llvm-commits mailing list