[PATCH] D113702: [lld-macho] Fix symbol relocs handling for compact unwind's functionAddress

[Vy Nguyen via Phabricator via llvm-commits]

oontvoo added a comment.

LG



================
Comment at: lld/MachO/InputFiles.cpp:930
+        // Skip if it is e.g. a weak def that didn't prevail.
+        if (sym->getFile() != this) {
+          ++it;
----------------
sorry, not 100% related to this, but  if the same input file is repeated on the command line, does the pointer equality hold true? or do they count as separate files?



================
Comment at: lld/test/MachO/weak-definition-gc.s:262
+## This was generated from compiling weak-sub-lsda.s above at rev a2404f11c77e
+## and then running it through `ld -r`. This converts a number of unwind-related
+## relocations from section- to symbol-based ones.
----------------
at this point is it worth implementing -r yet?
these yaml files are kind of annoying


================
Comment at: lld/test/MachO/weak-definition-gc.s:473-481
+    - EH_Frame1
+    - func.eh
+    - ''
+    - ''
+    - ''
+    - ''
+    - ''
----------------
do we need all these?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113702