On Sun, Oct 7, 2012 at 2:17 PM, Grzegorz Jablonski <span dir="ltr"><<a href="mailto:grzejabl@gmail.com" target="_blank">grzejabl@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
there is a bug <a href="http://llvm.org/bugs/show_bug.cgi?id=11806" target="_blank">http://llvm.org/bugs/show_bug.<u></u>cgi?id=11806</a> about -ast-print not being production quality. I am using clang to rewrite C++ source. Is anyone else using clang in that way? Some comments on the bug pages suggest, that nobody is using this feature.<br>
</blockquote><div><br></div><div>Yes, various people use Clang to rewrite C++ source, but mainly not by pretty-printing. Instead, targeted rewrites are performed on the original source file in each place that needs to be changed (see the Rewriter class). This has the advantage of preserving source formatting, comments, macros, header inclusions, etc. Generally speaking, pretty-printing isn't a good way to perform rewrites if you want to produce code for a person to maintain, because it loses all this information. Also, since the AST is intended to be essentially immutable, if you want to perform rewrites you are likely to find it easier to perform them textually rather than writing a pass to build a rewritten AST.</div>
<div><br></div><div>That said, making the AST pretty-printing more correct would be a good thing. Patches attached to bugs are not likely to get much attention; instead, you should mail the patches to <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a> and ask for them to be reviewed / committed.</div>
<div>-- </div><div>Richard</div></div>