<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jun 30, 2016 at 7:54 PM, Zachary Turner via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr">I submitted this yesterday in r274171.  I think this allows for a more intuitive way to structure the options of command line tools.  It's optional and if you do nothing llvm::cl will work as it always has for you.</div></blockquote><div><br></div><div>Perhaps you could define a default parameter value for the new parameter of getRegisteredOptions(...) ? That, or the code example given in the comments should be fixed.</div><div>I think a default parameter value = *cl::TopLevelSubCommand would mean that no code changes are necessary for downstream users (us ;).</div><div><br></div><div>cheers,</div><div>  Johan</div><div><br></div></div></div></div>