qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Richard Henderson <richard.henderson@linaro.org>
To: Taylor Simpson <tsimpson@quicinc.com>, qemu-devel@nongnu.org
Cc: philmd@linaro.org, peter.maydell@linaro.org, bcain@quicinc.com,
	quic_mathbern@quicinc.com, stefanha@redhat.com, ale@rev.ng,
	anjo@rev.ng, quic_mliebel@quicinc.com
Subject: Re: [PULL 00/11] Hexagon queue
Date: Fri, 21 Apr 2023 09:20:36 +0100	[thread overview]
Message-ID: <dd2f435f-19f6-b325-4594-89fda118eaa7@linaro.org> (raw)
In-Reply-To: <20230420032634.105311-1-tsimpson@quicinc.com>

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~


  parent reply	other threads:[~2023-04-21  8:21 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-20  3:26 [PULL 00/11] Hexagon queue Taylor Simpson
2023-04-20  3:26 ` [PULL 01/11] Hexagon (translate.c): avoid redundant PC updates on COF Taylor Simpson
2023-04-20  3:26 ` [PULL 02/11] Use f-strings in python scripts Taylor Simpson
2023-04-20  3:26 ` [PULL 03/11] Use black code style for " Taylor Simpson
2023-04-20  3:26 ` [PULL 04/11] Hexagon (target/hexagon) Remove redundant/unused macros Taylor Simpson
2023-04-20  3:26 ` [PULL 05/11] Hexagon (target/hexagon) Merge arguments to probe_pkt_scalar_hvx_stores Taylor Simpson
2023-04-20  3:26 ` [PULL 06/11] Hexagon (target/hexagon) Add overrides for count trailing zeros/ones Taylor Simpson
2023-04-20  3:26 ` [PULL 07/11] Hexagon (target/hexagon) Updates to USR should use get_result_gpr Taylor Simpson
2023-04-20  3:26 ` [PULL 08/11] Hexagon (tests/tcg/hexagon) Move HVX test infra to header file Taylor Simpson
2023-04-20  3:26 ` [PULL 09/11] Hexagon (target/hexagon) Remove unused slot variable in helpers Taylor Simpson
2023-04-20  3:26 ` [PULL 10/11] Hexagon (target/hexagon) Add overrides for cache/sync/barrier instructions Taylor Simpson
2023-04-20  3:26 ` [PULL 11/11] Hexagon (target/hexagon) Additional instructions handled by idef-parser Taylor Simpson
2023-04-21  8:20 ` Richard Henderson [this message]
2023-04-21 15:06   ` [PULL 00/11] Hexagon queue Taylor Simpson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=dd2f435f-19f6-b325-4594-89fda118eaa7@linaro.org \
    --to=richard.henderson@linaro.org \
    --cc=ale@rev.ng \
    --cc=anjo@rev.ng \
    --cc=bcain@quicinc.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=quic_mathbern@quicinc.com \
    --cc=quic_mliebel@quicinc.com \
    --cc=stefanha@redhat.com \
    --cc=tsimpson@quicinc.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).