diff options
| author | Jacob Pratt <jacob@jhpratt.dev> | 2025-01-31 00:25:36 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-01-31 00:25:36 -0500 |
| commit | dcf1134386cbfdd371269e8dcd1a051a6d1590b5 (patch) | |
| tree | 1aa6b4d510a1b55e95055028485c8cc6981864f2 /compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp | |
| parent | 1a5ebc9e52c452806ed3a32373e5ef92a168d466 (diff) | |
| parent | 022c0ce94787a76587fd39519f1358acd98fed49 (diff) | |
| download | rust-dcf1134386cbfdd371269e8dcd1a051a6d1590b5.tar.gz rust-dcf1134386cbfdd371269e8dcd1a051a6d1590b5.zip | |
Rollup merge of #136270 - nnethercote:rm-NamedVarMap, r=jackh726
Remove `NamedVarMap`. `NamedVarMap` is extremely similar to `ResolveBoundVars`. The former contains two `UnordMap<ItemLocalId, T>` fields (obscured behind `ItemLocalMap` typedefs). The latter contains two `SortedMap<ItemLocalId, T>` fields. We construct a `NamedVarMap` and then convert it into a `ResolveBoundVars` by sorting the `UnordMap`s, which is unnecessary busywork. This commit removes `NamedVarMap` and constructs a `ResolveBoundVars` directly. `SortedMap` and `NamedVarMap` have slightly different perf characteristics during construction (e.g. speed of insertion) but this code isn't hot enough for that to matter. A few details to note. - A `FIXME` comment is removed. - The detailed comments on the fields of `NamedVarMap` are copied to `ResolveBoundVars` (which has a single, incorrect comment). - `BoundVarContext::map` is renamed. - `ResolveBoundVars` gets a derived `Default` impl. r? `@jackh726`
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp')
0 files changed, 0 insertions, 0 deletions