[llvm] [LTO] Turn ImportListsTy into a proper class (NFC) (PR #106427)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 17 00:44:47 PDT 2024


nikic wrote:

We've encountered an issue when integrating this change into Rust. The problem is that this commit removed the possibility to fetch the import list without potentially modifying the DenseMap. Rust can read the ImportLists concurrently from multiple threads, and this may result in crashes, as reported at https://github.com/rust-lang/rust/pull/129749#issuecomment-2354417960. At least that's my current theory.

Rust was previously using `lookup()` instead of `operator[]`. Would it be possible to restore some form of read-only access to this data structure, whether via lookup or some other method?

https://github.com/llvm/llvm-project/pull/106427


More information about the llvm-commits mailing list