[PATCH] D110040: [lld-macho] Don't replace local personality symbol with LazySymbol

Vy Nguyen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 22 09:15:52 PST 2021


oontvoo added inline comments.


================
Comment at: lld/test/MachO/objc-uses-custom-personality.s:1
+## Tests that lld correctly resolves the custom personality referenced by objc code in an archive.
+
----------------
int3 wrote:
> the "objc" part seems tangential to the logic actually being tested, no? can we make the test non-objc-specific?
Actually I think this is actually quite essential :P because only ObjC archives has the semantic of "force-loading" when `-objC` is specified.
Regular C++ classes don't have this.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110040



More information about the llvm-commits mailing list