[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