[PATCH] D44695: [clang-format] Partially revert r322749, replacing array with DenseSet

Ben Hamilton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 20 12:10:13 PDT 2018


benhamilton added a comment.

I'm fine with whichever approach we use, but I'll defer to @krasimir here.

(I'm not fully sure why `llvm::DenseSet` is better than `std::unsorted_set`, but I'm sure you and @krasimir understand the subtleties better than I do.)


Repository:
  rC Clang

https://reviews.llvm.org/D44695





More information about the cfe-commits mailing list