[cfe-commits] r44338 - /cfe/trunk/AST/StmtPrinter.cpp

[Bill Wendling]

On Nov 26, 2007, at 6:50 PM, Chris Lattner wrote:
> On Nov 26, 2007, at 5:16 PM, Bill Wendling wrote:
>
>> Hi Ted,
>>
>> This probably doesn't make a huge difference, but you could pass the
>> various LLVM streams in by value. They are really just light-weight
>> wrappers around the STD::stream.
>
> I think Ted's patch is appropriate.  With this approach all the  
> apis can be declares as taking std::ostream, which is nicer than  
> LLVMostream.
>
Okay. I really don't have a huge preference. Just offering an  
alternative :-)

-bw