This is an archive of the discontinued LLVM Phabricator instance.

Differential D35153

Use DenseMap instead std::map for GVSummaryMapTy
ClosedPublic

Authored by danielcdh on Jul 7 2017, 3:04 PM.

Details

Reviewers
tejohnson
Commits
rG5f83d0e96bc9: Use DenseMap instead std::map for GVSummaryMapTy
rC307584: Use DenseMap instead std::map for GVSummaryMapTy
rL307584: Use DenseMap instead std::map for GVSummaryMapTy
Summary

Frontend change for https://reviews.llvm.org/D35148

Diff Detail

Event Timeline

danielcdh created this revision.Jul 7 2017, 3:04 PM
Herald added a subscriber: sanjoy. · View Herald TranscriptJul 7 2017, 3:04 PM
Harbormaster completed remote builds in B8068: Diff 105708.Jul 9 2017, 6:15 AM
tejohnson accepted this revision.Jul 9 2017, 7:00 AM

LGTM

This revision is now accepted and ready to land.Jul 9 2017, 7:00 AM
danielcdh closed this revision.Jul 10 2017, 1:31 PM

Revision Contents

PathSize
lib/
CodeGen/
2 lines

Diff 105708

lib/CodeGen/BackendUtil.cpp

Show First 20 Lines • Show All 992 Lines • ▼ Show 20 Lines
static void runThinLTOBackend(ModuleSummaryIndex *CombinedIndex, Module *M, static void runThinLTOBackend(ModuleSummaryIndex *CombinedIndex, Module *M,
const HeaderSearchOptions &HeaderOpts, const HeaderSearchOptions &HeaderOpts,
const CodeGenOptions &CGOpts, const CodeGenOptions &CGOpts,
const clang::TargetOptions &TOpts, const clang::TargetOptions &TOpts,
const LangOptions &LOpts, const LangOptions &LOpts,
std::unique_ptr<raw_pwrite_stream> OS, std::unique_ptr<raw_pwrite_stream> OS,
std::string SampleProfile, std::string SampleProfile,
BackendAction Action) { BackendAction Action) {
StringMap<std::map<GlobalValue::GUID, GlobalValueSummary *>> StringMap<DenseMap<GlobalValue::GUID, GlobalValueSummary *>>
ModuleToDefinedGVSummaries; ModuleToDefinedGVSummaries;
CombinedIndex->collectDefinedGVSummariesPerModule(ModuleToDefinedGVSummaries); CombinedIndex->collectDefinedGVSummariesPerModule(ModuleToDefinedGVSummaries);
setCommandLineOpts(CGOpts); setCommandLineOpts(CGOpts);
// We can simply import the values mentioned in the combined index, since // We can simply import the values mentioned in the combined index, since
// we should only invoke this using the individual indexes written out // we should only invoke this using the individual indexes written out
// via a WriteIndexesThinBackend. // via a WriteIndexesThinBackend.
▲ Show 20 Lines • Show All 259 Lines • Show Last 20 Lines