This is an archive of the discontinued LLVM Phabricator instance.

[lld-macho][docs] Cosmetic changes
ClosedPublic

Authored by int3 on Sep 12 2022, 10:59 AM.

Details

Summary
  1. Fixed rST hyperlink syntax
  2. Renamed LD64 -> ld64
  3. Moved up the -no_deduplicate section so it is right under the section talking about how our default dedup behavior differs; IMO it makes more sense to read them in that order
  4. De-bullet-listed some other sections so we have less whitespace in the rendered page
  5. Since the Mach-O LLD Port page has only one sub-page, don't render an entire toctree with just one item. Use a "See also" box instead.

Diff Detail

Event Timeline

int3 created this revision.Sep 12 2022, 10:59 AM
Herald added a reviewer: ributzka. · View Herald Transcript
Herald added projects: Restricted Project, Restricted Project. · View Herald Transcript
Herald added a subscriber: arphaman. · View Herald Transcript
int3 requested review of this revision.Sep 12 2022, 10:59 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 12 2022, 10:59 AM
int3 added a comment.Sep 12 2022, 12:01 PM

you could approve it ;)

int3 added a comment.Sep 12 2022, 12:24 PM

@ributzka are you sure you actually want to be added to every LLD-MachO diff

thevinster accepted this revision.Sep 12 2022, 12:47 PM
This revision is now accepted and ready to land.Sep 12 2022, 12:47 PM
This revision was automatically updated to reflect the committed changes.