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

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 20 12:20:59 PDT 2018


krasimir added a comment.

One way in which `DenseSet` is better is that it supports `StringRef`s - we don't have to define `hash`. Seems like the lack of this override in core LLVM suggests that `unordered_set` is not commonly used with `StringRef`s.


Repository:
  rC Clang

https://reviews.llvm.org/D44695





More information about the cfe-commits mailing list