[llvm-commits] [llvm] r142523 - in /llvm/trunk/lib/TableGen: TGParser.cpp TGParser.h

David A. Greene greened at obbligato.org
Wed Oct 19 14:34:03 PDT 2011


Eric Christopher <echristo at apple.com> writes:

>> Well, it's a tradeoff.  I think reviewers prefer smaller commits, so
>> that's what I aimed for.  These have been waiting for the 3.0 branch.
>
> smaller commits, yes, this many commits in about 30 seconds is pretty
> unfriendly though.

Hmm...ok, I can throttle next time.  But how much to throttle?  They're
going to be committed at some time.

Is the issue the need to review every commit before the next one
happens?  If so, progress is going to come to a screetching halt.

                              -Dave



More information about the llvm-commits mailing list