Re: [PULL 00/44] Hexagon update

[Richard Henderson]

On 5/12/23 22:46, Taylor Simpson wrote:
> The following changes since commit 278238505d28d292927bff7683f39fb4fbca7fd1:
>
>    Merge tag 'pull-tcg-20230511-2' ofhttps://gitlab.com/rth7680/qemu  into 
> staging (2023-05-11 11:44:23 +0100)
>
> are available in the Git repository at:
>
>    https://github.com/quic/qemu  tags/pull-hex-20230512-1
>
> for you to fetch changes up to a1c042e1cc4c1da209f7c3e04aec5e622c7bcdc0:
>
>    Hexagon (linux-user/hexagon): handle breakpoints (2023-05-12 14:43:14 -0700)
>
> ----------------------------------------------------------------
> This PR can be broken down into the following parts
> - Add support for new architecture versions v68/v69/v71/v73
> - Short-circuit writes to temporaries when packet semantics permit this
> - Move bookkeeping items from CPUHexagonState to DisasContext
> - Correct '-cpu help' output and handling of unknown Hexagon versions
> - Enable LLDB debugging
> - Miscellaneous fixes and improvements

Please always test 32-bit host.  You have a CI failure:

https://gitlab.com/qemu-project/qemu/-/jobs/4279656830#L873

/builds/qemu-project/qemu/target/hexagon/idef-parser/idef-parser.lex: In 
function 'yylex':
/builds/qemu-project/qemu/target/hexagon/idef-parser/idef-parser.lex:435:38: error: format 
'%lx' expects argument of type 'long unsigned int', but argument 2 has type 'int64_t' {aka 
'long long int'} [-Werror=format=]
  435 |                               printf("value = 0x%lx\n", value);
      |                                      ^~~~~~~~~~~~~~~~~  ~~~~~
      |                                                         |
      |                                                         int64_t {aka 
long long int}



r~