[llvm-commits] [llvm] r134907 - /llvm/trunk/utils/TableGen/

David A. Greene greened at obbligato.org
Tue Jul 12 11:06:04 PDT 2011


Jakob Stoklund Olesen <stoklund at 2pi.dk> writes:

>> If you're not getting a response, please ping the patch like everyone else.  Ignoring policy is not the solution.
>
>
> David's request for review:	Fri  9:41 AM PDT
> My reply to David:		Mon 10:11 AM PDT
> David's committing r134907:	Mon 11:25 AM PDT

My timeline was off.  I apologize.  But it still took a very long time
to get any sort of response.  I have waited days in the past.  That
simply won't work in a fast-moving project like LLVM.  I had to
constantly update this patch over the period between the review request
and the actual commit due to changes in LLVM.

Can we institute some kind of deadline for review request reponses?
That way at least the submitter won't be stuck forever.

                                -Dave



More information about the llvm-commits mailing list