> -----Original Message-----
> From: Richard Henderson <richard.hender...@linaro.org>
> Sent: Friday, April 21, 2023 3:21 AM
> To: Taylor Simpson <tsimp...@quicinc.com>; qemu-devel@nongnu.org
> Cc: phi...@linaro.org; peter.mayd...@linaro.org; Brian Cain
> <bc...@quicinc.com>; Matheus Bernardino (QUIC)
> <quic_mathb...@quicinc.com>; stefa...@redhat.com; a...@rev.ng;
> a...@rev.ng; Marco Liebel (QUIC) <quic_mlie...@quicinc.com>
> Subject: Re: [PULL 00/11] Hexagon queue
>
> On 4/20/23 05:26, Taylor Simpson wrote:
> > The following changes since commit
> 60ca584b8af0de525656f959991a440f8c191f12:
> >
> Fails CI job clang-user:
>
> target/hexagon/idef-generated-emitter.indented.c:32174:39: error: implicit
> conversion from 'int64_t' (aka 'long') to 'int32_t' (aka 'int') changes value
> from
> 2147483648 to
> -2147483648 [-Werror,-Wconstant-conversion]
>
> int32_t qemu_tmp_3 = ((int64_t) 1ULL) << ((int64_t) 31ULL);
> ~~~~~~~~~~ ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~
>
> https://gitlab.com/qemu-project/qemu/-/jobs/4153502279#L1140
My apologies.
This is the patch causing the error
Hexagon (target/hexagon) Additional instructions handled by idef-parser
I'll have to work with the rev.ng guys to get it fixed.
For now, I'll remove this patch and resubmit the pull request.
Thanks,
Taylor
