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

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 03:52:14 PDT 2019


jhenderson added a comment.

Hi @dsprenkels,

Thanks for the patch. I've just got back from a couple of weeks off, and have a lot to catch up on, but I'll get back to reviewing this in due course (it may be a few days, depending on how things go). In the meantime, you might want to look at who else has recently modified this file and add them as reviewers too.

You also should add one or more tests to show that the bug this issue fixes has been fixed.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D69501





More information about the llvm-commits mailing list