<div class="gmail_extra"><div class="gmail_quote">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>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">http://article.gmane.org/gmane.comp.compilers.llvm.cvs/117923/match=alexfh</a></div>
<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 class="im"><br>
On Fri, Jul 27, 2012 at 3:33 PM, Alexander Kornienko <<a href="mailto:alexfh@google.com">alexfh@google.com</a>> wrote:<br>
</div><div class=""><div class="h5">> 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><br>-- </div><div class="gmail_extra">Best regards,</div><div class="gmail_extra">Alexander Kornienko</div>