Seems reasonable, I'll take a look later <br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 1, 2016 at 8:00 AM Johan Engelen <<a href="mailto:jbc.engelen@gmail.com">jbc.engelen@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><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>
</blockquote></div>