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

Derek Schuff via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 1 14:12:51 PST 2024


dschuff wrote:

We just discussed live and decided that we don't need to combine symbol info from the export and name sections, that the names from the name section can take precedence. However I realized that we will still need to fix how these are stored because the name section does not declare the total number of names up front (instead it has subsections for each name kind). So assuming we want symbols for both function and data section names, we can't allocate the vectors up front. So we'll either need to change the data representation as this PR does, or do a preparse step to skip ahead in the section and find the total name count before reading the names.

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


More information about the llvm-commits mailing list