MaskRay added inline comments.

================
Comment at: llvm/tools/llvm-ar/llvm-ar.cpp:74
+      << "  -U                    - Use actual timestamps and uids/gids\n"
+      << "  -X{32|64|32_64|any}       - Specifies which archive symbol tables "
+         "should be generated if they do not already exist (AIX OS only)\n";
----------------
"Specify". The convention is to use imperative sentences. Also, does the `-` 
align with `-` above?

(The capitalization is a bit inconsistent. For existing llvm-ranlib options, 
capitalization is more common.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142660

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to