[PATCH] D68998: [docs][llvm-ar] Update llvm-ar command guide

[Jordan Rupprecht via Phabricator via llvm-commits]

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

Thanks! Please wait for Rui to take another look, but this looks good enough to me.



================
Comment at: llvm/docs/CommandGuide/llvm-ar.rst:324
 
-ar(1)
+:manpage:`ar(1)`
----------------
gbreynoo wrote:
> rupprecht wrote:
> > This is a dead link for me - https://llvm.org/docs/CommandGuide/ar.html
> This was a carry over from the current guide. Should a link to the gnu-ar command guide be put here instead or would it be best to remove the section all together? 
I think we can just remove it. All the other docs in the "GNU binutils replacements" sections only have a "See also" section if they refer to other llvm tools (e.g. llvm-objcopy refers to llvm-strip). We already advertise heavily in the intro that this is a suitable replacement for GNU ar.


================
Comment at: llvm/docs/CommandGuide/llvm-ar.rst:41
+ 
+ [l] - ignored in :program:`ar` 
 
----------------
Since GNU ar seems to also accept but ignore this arg, is it worth mentioning at all? Or are there other archiver programs that do use it?


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

https://reviews.llvm.org/D68998