[lld] [lld][WebAssembly] Match the ELF linker in transitioning away from archive indexes. (PR #78658)

Sam Clegg via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 19 15:41:18 PST 2024


================
@@ -495,26 +495,21 @@ class UndefinedTag : public TagSymbol {
 // and the lazy. We represent that with a lazy symbol with a weak binding. This
 // means that code looking for undefined symbols normally also has to take lazy
 // symbols into consideration.
-class LazySymbol : public Symbol {
+class LazyObject : public Symbol {
----------------
sbc100 wrote:

It seems like this was always just called LazyObject on the ELF side.

I'm not sure why `LazyObject` was used.. `LazySymbol` seems more descriptive.  I think maybe I will just revert this rename to keep this change small . We can always followup with a rename-only PR afterwards.

@MaskRay any ideas on this naming?  Does `LazyObject` make sense for some reason that I am missing?

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


More information about the llvm-commits mailing list