c9b45e6010
Change several HashMaps to IndexMap to improve incremental hashing performance Stable hashing hash maps in incremental mode takes a lot of time, especially for some benchmarks like `clap`. As noted by `@Mark-Simulacrum` [here](https://github.com/rust-lang/rust/pull/89404#issuecomment-950043892), this cost could be reduced by replacing some hash maps by indexmaps. I gathered some statistics and found several hash maps that took a lot of time to hash and replaced them by indexmaps. However, in order for this to work, we need to make sure that these indexmaps have deterministic insertion order. These three are used only in visitors as far as I can see, which seems deterministic. Can we enforce this somehow? Or should some explaining comment be included for these maps? |
||
|---|---|---|
| .. | ||
| codegen_fn_attrs.rs | ||
| dependency_format.rs | ||
| exported_symbols.rs | ||
| lang_items.rs | ||
| limits.rs | ||
| mod.rs | ||
| privacy.rs | ||
| region.rs | ||
| resolve_lifetime.rs | ||
| stability.rs | ||