[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:26:54 PDT 2018


benhamilton added a comment.

> One way in which DenseSet is better is that it supports StringRefs - 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 StringRefs.

Makes sense, thanks.


Repository:
  rC Clang

https://reviews.llvm.org/D44695





More information about the cfe-commits mailing list