After D42111, the executor framework no longer deduplicate tool results.
|109 ↗||(On Diff #129984)|
nit: this comment could be tightened up a bit:
// FIXME(ioeric): Merge occurrences, rather than just dropping all but one. // Definitions and forward declarations have the same key and may both have information. // Usage count will need to be aggregated across occurrences, too.
|113 ↗||(On Diff #129984)|
nit: UniqueSymbols? "Reduce" makes sense from an MR perspective but there's not enough context.
|116 ↗||(On Diff #129984)|
picking the longest string might be better than random, but I'm not sure if it's enough to be worthwhile.