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

[Owen Reynolds via Phabricator via llvm-commits]

gbreynoo marked 2 inline comments as done.
gbreynoo added inline comments.


================
Comment at: llvm/docs/CommandGuide/llvm-ar.rst:41
+ 
+ [l] - ignored in :program:`ar` 
 
----------------
rupprecht wrote:
> 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?
I know this is mentioned in the help text, but as this effects Command Line I think it belongs here. I can't tell you which archiver used this option originally. 


================
Comment at: llvm/docs/CommandGuide/llvm-ar.rst:106
 
-d
+.. option:: d [NT]
+
----------------
MaskRay wrote:
> `P` can affect `d`.
> 
> Can `T` affect the `d` operation?
As a general modifier I haven't been explicit with which commands use `P`, this is consistent with the other general modifiers. Regarding `T` with `D`, current llvm-ar behaviour will convert thin to full archives if `T` is not included with a write command. I'm not sure if this is the behaviour we want. 


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

https://reviews.llvm.org/D68998