Update: implemented -ast-list option and added tests for all dump-related options in clang-check.<div><br><div class="gmail_quote">On Fri, Jul 27, 2012 at 3:48 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_extra"><div class="gmail_quote"><div class="im">On Fri, Jul 27, 2012 at 3:40 PM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Dependent on somebody confirming that:<br>
a) using cl::opt as non-static variables is a supported use case<br></blockquote></div><div>As for this, there's a unit-test waiting to be reviewed on the llvm-commits list: <a href="http://article.gmane.org/gmane.comp.compilers.llvm.cvs/117923/match=alexfh" target="_blank">http://article.gmane.org/gmane.comp.compilers.llvm.cvs/117923/match=alexfh</a></div>
<div class="im">
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
b) we don't have a better option to add -extra-clang-flags that would<br>
allow us to also specify -ast-dump*<br>
... this looks good to me.<br>
<br>
Chandler, opinions on either of those?<br>
<div><br>
On Fri, Jul 27, 2012 at 3:33 PM, Alexander Kornienko <<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>> wrote:<br>
</div><div><div>> Hi all,<br>
><br>
> This patch adds -ast-dump, -ast-print and -ast-dump-filter options to<br>
> clang-check. It uses the existing implementation of these options and only<br>
> makes them available to use in clang-check. A test is included.<br>
><br>
> Please, review this patch. Thanks!<br>
><br>
> --<br>
> Best regards,<br>
> Alexander Kornienko<br>
</div></div></blockquote></div></div><span class="HOEnZb"><font color="#888888"><br>-- </font></span></div><span class="HOEnZb"><font color="#888888"><div class="gmail_extra">Best regards,</div><div class="gmail_extra">Alexander Kornienko</div>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Best regards,</div><div>Alexander Kornienko</div><br>
</div>