[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