[PATCH] D122904: [lld/mac] Tweak a few comments

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 1 07:14:25 PDT 2022


thakis created this revision.
thakis added a reviewer: lld-macho.
Herald added projects: lld-macho, All.
thakis requested review of this revision.

Addresses review feedback I had missed on https://reviews.llvm.org/D122624

No behavior change.


https://reviews.llvm.org/D122904

Files:
  lld/MachO/ConcatOutputSection.cpp
  lld/test/MachO/arm64-thunk-visibility.s


Index: lld/test/MachO/arm64-thunk-visibility.s
===================================================================
--- lld/test/MachO/arm64-thunk-visibility.s
+++ lld/test/MachO/arm64-thunk-visibility.s
@@ -1,7 +1,7 @@
 # REQUIRES: aarch64
 
 # foo.s and bar.s both contain TU-local symbols (think static function)
-# with the same name, and both need a thunk..  This tests that ld64.lld doesn't
+# with the same name, and both need a thunk.  This tests that ld64.lld doesn't
 # create a duplicate symbol for the two functions.
 
 # Test this both when the TU-local symbol is the branch source or target,
Index: lld/MachO/ConcatOutputSection.cpp
===================================================================
--- lld/MachO/ConcatOutputSection.cpp
+++ lld/MachO/ConcatOutputSection.cpp
@@ -84,15 +84,17 @@
 //   distant call sites might be unable to reach the same thunk, so multiple
 //   thunks are necessary to serve all call sites in a very large program. A
 //   thunkInfo stores state for all thunks associated with a particular
-//   function: (a) thunk symbol, (b) input section containing stub code, and
-//   (c) sequence number for the active thunk incarnation. When an old thunk
-//   goes out of range, we increment the sequence number and create a new
-//   thunk named <FUNCTION>.thunk.<SEQUENCE>.
+//   function:
+//     (a) thunk symbol
+//     (b) input section containing stub code, and
+//     (c) sequence number for the active thunk incarnation.
+//   When an old thunk goes out of range, we increment the sequence number and
+//   create a new thunk named <FUNCTION>.thunk.<SEQUENCE>.
 //
-// * A thunk consists of (a) a Defined symbol pointing to
-//   (b) an InputSection holding machine instructions (similar to a MachO
-//   stub), and (c) Reloc(s) that reference the real function for fixing-up
-//   the stub code.
+// * A thunk consists of
+//     (a) a Defined symbol pointing to
+//     (b) an InputSection holding machine code (similar to a MachO stub), and
+//     (c) relocs referencing the real function for fixing up the stub code.
 //
 // * std::vector<InputSection *> MergedInputSection::thunks: A vector parallel
 //   to the inputs vector. We store new thunks via cheap vector append, rather


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D122904.419749.patch
Type: text/x-patch
Size: 2237 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220401/ce8ad04d/attachment.bin>


More information about the llvm-commits mailing list