[lld] [llvm] [Object][Wasm] Move WasmSymbolInfo directly into WasmSymbol (NFC) (PR #80219)

Derek Schuff via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 31 16:01:28 PST 2024


dschuff wrote:

@sbc100 this is a prototype PR to move the `WasmSymbolInfo`s from their own vector on the `WasmLinkingData` directly into the `WasmSymbol` object. Removing the const-ref to an external object allows the vector of `WasmSymbol`s to be safely expanded/reallocated (which we'd need to do if we want to synthesize symbols from export and name sections together). 
It can maybe be further refined, but mostly I'm wondering if you recall why the organization was this way in the first place. The `WasymSymbolInfo`s never seem to be accessed other than via the `WasmSymbol` itself, so this ended up being a very minimal change.

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


More information about the llvm-commits mailing list