[PATCH] D69501: [CommandLine] Add inline ArgName printing

Daan Sprenkels via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 29 08:21:11 PDT 2019


dsprenkels updated this revision to Diff 226908.
dsprenkels marked an inline comment as done.
dsprenkels added a comment.

Fixed formatting using `clang-format`.

> As an aside, once you've addressed a review comment, it's helpful if you select the "Done" button for each of those comments. You can do this prior to uploading the diff and they will get submitted when you upload too.

As you might have noticed, this is my first review on Phabricator. Thanks for your patience. :)


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

https://reviews.llvm.org/D69501

Files:
  llvm/lib/Support/CommandLine.cpp
  llvm/unittests/Support/CommandLineTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69501.226908.patch
Type: text/x-patch
Size: 4454 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191029/f5836275/attachment.bin>


More information about the llvm-commits mailing list