[PATCH] D51559: [LLD][COFF] Cleanup error messages / add more coverage tests
Rui Ueyama via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 3 18:03:46 PDT 2018
ruiu added inline comments.
================
Comment at: COFF/PDB.cpp:168
/// Cached to prevent repeated load attempts.
- std::set<GUID> MissingTypeServerPDBs;
+ std::map<GUID, std::string> MissingTypeServerPDBs;
};
----------------
We generally don't use std::map because it's slower than llvm::DenseMap and in most cases we don't need an ordered map (std::map is ordered). But this use may be okay because original code uses std::set. Or, we don't need to care about it at all if the number of data inserted into this map/set is small. Zach?
Repository:
rLLD LLVM Linker
https://reviews.llvm.org/D51559
More information about the llvm-commits
mailing list