[PATCH] D98686: [lld:elf] Weaken the requirement for a computed binding to be STB_LOCAL

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

> b.cpp being part of the link causes the vtbl for _ZTV3Cat to be STB_GLOBAL since it's external.

Can you elaborate what happens? b.o has only one symbol `_Z14doThingWithCatP3Cat`. There is no vtable symbol...

Only `canBeVersioned` symbols can have a `versionId` value not equal to `VER_NDX_GLOBAL`. How does the `isDefined() -> !isLazy()` affect behaviors?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98686/new/

https://reviews.llvm.org/D98686