This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Use StringMap for ClangTidyOptions::OptionsMap
ClosedPublic

Authored by njames93 on Jul 29 2020, 9:23 AM.

Details

Summary

Ordering of options isn't important so an llvm::StringMap is a much better container for this purpose.

Diff Detail

Event Timeline

njames93 created this revision.Jul 29 2020, 9:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 29 2020, 9:23 AM
njames93 requested review of this revision.Jul 29 2020, 9:23 AM
njames93 updated this revision to Diff 281639.Jul 29 2020, 9:27 AM

Missing call to getValue()

gribozavr2 accepted this revision.Jul 29 2020, 10:40 AM
This revision is now accepted and ready to land.Jul 29 2020, 10:40 AM