This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Use DenseSet<SourceLocation> in UpgradeDurationConversionsCheck, NFCI
ClosedPublic

Authored by miyuki on Jan 13 2021, 7:20 AM.

Details

Summary

This change replaces unordered_set<unsigned> (which used to store
internal representation of SourceLocation-s) with
DenseSet<SourceLocation> (which stores SourceLocation-s directly).

Diff Detail

Event Timeline

miyuki created this revision.Jan 13 2021, 7:20 AM
miyuki requested review of this revision.Jan 13 2021, 7:20 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 13 2021, 7:20 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Eugene.Zelenko added a project: Restricted Project.
aaron.ballman accepted this revision.Jan 14 2021, 4:35 AM

LGTM, thank you!

This revision is now accepted and ready to land.Jan 14 2021, 4:35 AM
njames93 accepted this revision.Jan 14 2021, 4:37 AM

Seems reasonable, LGTM.