RE: [PULL 00/11] Hexagon queue

[Taylor Simpson]

> -----Original Message-----
> From: Richard Henderson <richard.henderson@linaro.org>
> Sent: Friday, April 21, 2023 3:21 AM
> To: Taylor Simpson <tsimpson@quicinc.com>; qemu-devel@nongnu.org
> Cc: philmd@linaro.org; peter.maydell@linaro.org; Brian Cain
> <bcain@quicinc.com>; Matheus Bernardino (QUIC)
> <quic_mathbern@quicinc.com>; stefanha@redhat.com; ale@rev.ng;
> anjo@rev.ng; Marco Liebel (QUIC) <quic_mliebel@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