[llvm-commits] [llvm] r77397 - in /llvm/trunk: include/llvm/Support/FormattedStream.h lib/Support/FormattedStream.cpp

[Evan Cheng]

On Jul 29, 2009, at 9:03 AM, David Greene wrote:

> On Tuesday 28 July 2009 23:48, Evan Cheng wrote:
>> Ok. David, I'm going to revert AsmWriterEmitter.cpp back to 74742 for
>> now. Please re-commit as you have fixed the efficiency problem.
>
> As you know, I can't SEE the efficiency problem.  I'm flying blind.
>
> And I was following Chris' suggestion to commit the patch I had with  
> the
> understanding that more fixes were coming.
>
> Honestly, this is getting very frustrating.  Aren't we in development
> right now?  Regressions are going to happen.  This kind of compile  
> time
> regression shouldn't be a huge problem especially when we have a  
> solution.

David,

This is a significant regression that hurts everyone. It slows down  
nightly testing significantly.

Dan was able to reproduce the regression on a Linux box. Folks do try  
to use other platforms to identify problems. Please do so.

Evan

>
> Oh well, guess I have to spend more time re-doing what I've already  
> done.
>
>                                  -Dave
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits