[PATCH] D106767: [lld/mac] Add support for segment$start$ and segment$end$ symbols

[Jez Ng via Phabricator via llvm-commits]

int3 accepted this revision.
int3 added a comment.
This revision is now accepted and ready to land.

Nice!



================
Comment at: lld/test/MachO/start-end.s:189-198
+# SEG-NOT: segment$start$__TEXT
+# SEG-NOT: segment$end$__TEXT
+# SEG-NOT: segment$start$__FOO
+# SEG-NOT: segment$end$__FOO
+# SEG-NOT: segment$start$__BAZ
+# SEG-NOT: segment$end$__BAZ
+# SEG-NOT: segment$start$__WHAT
----------------
maybe `--implicit-check-not=segment$` would be terser?


================
Comment at: lld/test/MachO/start-end.s:399-409
+  # (These two lines make ld64 crash linking the .o file.
+  # The rest of the test works with ld64 too.)
+  movq segment$start$__WHAT at GOTPCREL(%rip), %rax
+  movq segment$end$__WHAT at GOTPCREL(%rip), %rbx
+
+  # References to non-existing segments create that segment,
+  # without any sections in it.
----------------
nit: standardize whether we are using # or ##


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

https://reviews.llvm.org/D106767