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

Bill Wendling isanbard at gmail.com
Tue Nov 27 00:08:01 PST 2007

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 :-)


More information about the cfe-commits mailing list