diff options
| author | Camelid <camelidcamel@gmail.com> | 2021-02-22 15:43:43 -0800 |
|---|---|---|
| committer | Camelid <camelidcamel@gmail.com> | 2021-03-05 19:40:54 -0800 |
| commit | 4c51a66d674c32b3d8eeebcd1ae276611e9f4c12 (patch) | |
| tree | d599b56312a4e36256ce2b9c56eac444dbdcd59c /compiler/rustc_codegen_llvm/src/builder.rs | |
| parent | 3bc879e76bb428d42831179829606dede0c84f94 (diff) | |
| download | rust-4c51a66d674c32b3d8eeebcd1ae276611e9f4c12.tar.gz rust-4c51a66d674c32b3d8eeebcd1ae276611e9f4c12.zip | |
Don't share `id_map` and `deref_id_map`
All the tests passed, so it doesn't seem they need to be shared. Plus they should be item/page-specific. I'm not sure why they were shared before. I think the reason `id_map` worked as a shared value before is that it is cleared before rendering each item (in `render_item`). And then I'm guessing `deref_id_map` worked because it's a hashmap keyed by `DefId`, so there was no overlap (though I'm guessing we could have had issues in the future). Note that `id_map` currently still has to be cleared because otherwise child items would inherit the `id_map` of their parent. I'm hoping to figure out a way to stop cloning `Context`, but until then we have to reset `id_map`.
Diffstat (limited to 'compiler/rustc_codegen_llvm/src/builder.rs')
0 files changed, 0 insertions, 0 deletions
