[PATCH] D53332: Access ADDLIB in llvm-ar via command line

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 25 11:26:37 PDT 2018


MaskRay added inline comments.


================
Comment at: tools/llvm-ar/llvm-ar.cpp:681
+    assert(Operation == QuickAppend);
+    for (auto &Member : Members) {
+      addLibMember(Ret, Member);
----------------
I think `{` `}` is redundant as there is another case below that does not add `{` `}` for simple statement like this..


Repository:
  rL LLVM

https://reviews.llvm.org/D53332





More information about the llvm-commits mailing list