Re: [PATCH] is_promisor_object(): walk promisor packs in pack-order

From: Jonathan Tan <jonathantanmy@google.com>
Date: Jun 17, 2022, 19:50 UTC
Message-ID: <20220617195028.2698928-1-jonathantanmy@google.com>
In-Reply-To: <YqrTsbXbEjx0Pabn@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

Show 8 quoted lines

> On Thu, Jun 16, 2022 at 02:07:41AM -0400, Jeff King wrote:
> 
> > Those rev-lists run in 1.7s and 224s respectively. Ouch!
> 
> Even though I expected the second one to be slow, I was shocked at just
> how slow it was. The patch below speeds it up by a factor of 2, and I
> think is worth applying separately, regardless of anything else being
> discussed in this thread.

Thanks for the patch and the benchmark. This makes sense to me.