Re: [PULL 00/11] Hexagon queue

[Richard Henderson]

On 4/20/23 05:26, Taylor Simpson wrote:
> The following changes since commit 60ca584b8af0de525656f959991a440f8c191f12:
>
>    Merge tag 'pull-for-8.0-220323-1' of https://gitlab.com/stsquad/qemu into 
> staging (2023-03-22 17:58:12 +0000)
>
> are available in the Git repository at:
>
>    https://github.com/quic/qemu tags/pull-hex-20230419
>
> for you to fetch changes up to 073de8caa8f9013b4dd8613df29d15d7c42aa8f4:
>
>    Hexagon (target/hexagon) Additional instructions handled by idef-parser 
> (2023-04-19 20:13:20 -0700)
>
> ----------------------------------------------------------------
> Hexagon update
>
> ----------------------------------------------------------------
> Marco Liebel (2):
>        Use f-strings in python scripts
>        Use black code style for python scripts
>
> Matheus Tavares Bernardino (1):
>        Hexagon (translate.c): avoid redundant PC updates on COF
>
> Taylor Simpson (8):
>        Hexagon (target/hexagon) Remove redundant/unused macros
>        Hexagon (target/hexagon) Merge arguments to probe_pkt_scalar_hvx_stores
>        Hexagon (target/hexagon) Add overrides for count trailing zeros/ones
>        Hexagon (target/hexagon) Updates to USR should use get_result_gpr
>        Hexagon (tests/tcg/hexagon) Move HVX test infra to header file
>        Hexagon (target/hexagon) Remove unused slot variable in helpers
>        Hexagon (target/hexagon) Add overrides for cache/sync/barrier 
> instructions
>        Hexagon (target/hexagon) Additional instructions handled by idef-parser

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


r~