Hi Chris,<div><br></div><div>Please find the attached that cleans up the places that I found from greping through the source. Note that I just finished up things that I found in header files as you said you were going to go through the .cpp files. I built with Release+Asserts after making these changes and ran the test suite without issue.</div>
<div><br></div><div>FYI, I'm going to move onto ArrayRef next and will clean things up in the cpp files as I go through if I notice anything.</div><div><br></div><div>Thanks,</div><div>Jon<br><br><div class="gmail_quote">
On Fri, Jul 22, 2011 at 3:31 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.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="im"><br>
On Jul 22, 2011, at 11:58 AM, Mulder, Jonathan wrote:<br>
<br>
> Hi Chris,<br>
><br>
> This may be a silly question, but is there a reason why these changes were restricted to the header files? Or is it just that you haven't gotten around to making the changes in the .cpp files?<br>
<br>
</div>Um, that looks like a 'find' failure. I certainly intended .cpp files to be changed as well, I'll take care of that.<br>
<div class="im"><br>
> Also, I notice that there are still some forward declarations of raw_ostream in some header files, for instance "include/clang/AST/PrettyPrinter.h". Are these left as in-file forward declarations versus including "LLVM.h" because the utility of leaving the forward declaration in-file and tying out the using statement versus including "LLVM.h" isn't sufficient for being able to change one case of "llvm::raw_stream" to "raw_ostream" in the header file? Or is this - like the last question - just a case of not having gotten here yet? Or have I just completely missed the boat here? :)<br>

<br>
</div>These I missed, if you'd like to tackle them, I'd appreciate it!  Thanks,<br>
<font color="#888888"><br>
-Chris<br>
<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Jon Mulder<div>C 847.899.3664</div><div>E <a href="mailto:mulderje@muohio.edu" target="_blank">mulderje@muohio.edu</a></div><br>
</div>