On Mon, Aug 21, 2023 at 06:51:00PM +0200, Willy Tarreau wrote:
> Hi Tim, Peter,
>
> On Mon, Aug 21, 2023 at 05:36:58PM +0200, Tim Düsterhus wrote:
> > Peter,
> >
> > On 8/21/23 17:23, PR Bot wrote:
> > > Patch title(s):
> > > Fix build error on ppc64le: invalid parameter combination for AltiVec
> > > intrinsic __builtin_vec_ld
> > >
> >
> > xxhash.h should not be modified, because it is an external library. However
> > it appears that this might already be fixed upstream [1], thus a new version
> > of xxhash.h could probably be bundled (Ccing Willy).
>
> I'm not surprised at all, that's what I thought as well, and I planned
> to just update it again or to backport the fix. Just didn't yet have the
> time to work on this.
I had a look and we're at version 0.8.1. This issue was fixed some time
after 0.8.1 with this commit:
15ce80f ("Fix compilation on RHEL 7 ppc64le (gcc 4.8)")
It turned the definition to an unsigned int* instead. So I'll update it
to 0.8.2 and that should be OK.
Cheers,
Willy
