Change Details

The ordering is not needed, and std::unordered_map is faster (and std::hash for ConstString is trivial). I can measure time spent in the SaveToCache() calls reduced to ~50% during LLDB startup (and a ~25% reduction of the total startup cost).

The ordering is not needed, and DenseMap is faster. I can measure time spent in the SaveToCache() calls reduced to ~40% during LLDB startup (and the total startup cost reduced to ~70%).

The ordering is not needed, and std::unordered_mDenseMap is faster (and std::hash for ConstString is trivial)r. I can measure time spent in the SaveToCache() calls reduced to ~50%~40% during LLDB startup (and a ~25% reduction of the total startup costt reduced to ~70%).