On Tue, Jun 13, 2023 at 07:07:38PM -0700, Junio C Hamano wrote:

Eric Sunshine [off-list ref] writes:

quoted

Is this the same issue Peff reported[1]? His proposed solution was a
bit different.

[1]: https://lore.kernel.org/git/20230519005447.GA2955320@coredump.intra.peff.net/ (local)

Ah, I should have checked before merging the topic in 'next'.

I only found it after it was in 'next'. :)

It originally did not cause problems, but the switch to clang started
triggering it in CI (it was this case that motivated me to send those
patches CI patches).

I was not following the topic closely, but I think there is a v10 of
this series anyway, so you may want to eject it from 'next' in the
meantime (I had assumed you were going to do so as part of the
post-release rewind, but it looks like that happened already and it got
re-merged).

-Peff