[Qemu-commits] [qemu/qemu] c23b57: Hexagon (target/hexagon) probe the stores in a pac...

[Richard Henderson]

  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: c23b5764e79f3951e98160faf6e97284453c4056
      
https://github.com/qemu/qemu/commit/c23b5764e79f3951e98160faf6e97284453c4056
  Author: Taylor Simpson <tsimpson@quicinc.com>
  Date:   2021-10-06 (Wed, 06 Oct 2021)

  Changed paths:
    M target/hexagon/helper.h
    M target/hexagon/op_helper.c
    M target/hexagon/translate.c
    M tests/tcg/hexagon/Makefile.target
    A tests/tcg/hexagon/hex_sigsegv.c

  Log Message:
  -----------
  Hexagon (target/hexagon) probe the stores in a packet at start of commit

When a packet has 2 stores, either both commit or neither commit.
At the beginning of gen_commit_packet, we check for multiple stores.
If there are multiple stores, call a helper that will probe each of
them before proceeding with the commit.

Note that we don't call the probe helper for packets with only one
store.  Therefore, we call process_store_log before anything else
involved in committing the packet.

We also fix a typo in the comment in process_store_log.

Test case added in tests/tcg/hexagon/hex_sigsegv.c

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Taylor Simpson <tsimpson@quicinc.com>
Message-Id: <1633036599-7637-1-git-send-email-tsimpson@quicinc.com>


  Commit: f844f745a81a8b8dc7f85eaa3fe6a3bb880afaff
      
https://github.com/qemu/qemu/commit/f844f745a81a8b8dc7f85eaa3fe6a3bb880afaff
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-10-06 (Wed, 06 Oct 2021)

  Changed paths:
    M target/hexagon/macros.h

  Log Message:
  -----------
  target/hexagon: Remove unused TCG temporary from predicated loads

The gen_pred_cancel() function, introduced in commit a646e99cb90
(Hexagon macros) doesn't use the 'one' TCG temporary; remove it.

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Taylor Simpson <tsimpson@quicinc.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20211003004750.3608983-2-f4bug@amsat.org>


  Commit: 23803bbe524c34d5725508f169a0a23f652e6584
      
https://github.com/qemu/qemu/commit/23803bbe524c34d5725508f169a0a23f652e6584
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-10-06 (Wed, 06 Oct 2021)

  Changed paths:
    M target/hexagon/gen_tcg.h
    M target/hexagon/gen_tcg_funcs.py
    M target/hexagon/genptr.c
    M target/hexagon/macros.h
    M target/hexagon/translate.c

  Log Message:
  -----------
  target/hexagon: Use tcg_constant_*

Replace uses of tcg_const_* with the allocate and free close together.

Inspired-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Taylor Simpson <tsimpson@quicinc.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20211003004750.3608983-3-f4bug@amsat.org>


  Commit: ca61fa4b803e5d0abaf6f1ceb690f23bb78a4def
      
https://github.com/qemu/qemu/commit/ca61fa4b803e5d0abaf6f1ceb690f23bb78a4def
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-10-06 (Wed, 06 Oct 2021)

  Changed paths:
    M target/hexagon/gen_tcg.h
    M target/hexagon/gen_tcg_funcs.py
    M target/hexagon/genptr.c
    M target/hexagon/helper.h
    M target/hexagon/macros.h
    M target/hexagon/op_helper.c
    M target/hexagon/translate.c
    M tests/tcg/hexagon/Makefile.target
    A tests/tcg/hexagon/hex_sigsegv.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/quic/tags/pull-hex-20211006' into 
staging

Change from Philippe - Use tcg_constant_*
Change from Philippe - Remove unused TCG temp
Change from Taylor - Probe the stores in a packet at start of commit

# gpg: Signature made Wed 06 Oct 2021 08:44:13 AM PDT
# gpg:                using RSA key 7B0244FB12DE4422
# gpg: Good signature from "Taylor Simpson (Rock on) <tsimpson@quicinc.com>" 
[marginal]
# gpg: WARNING: This key is not certified with sufficiently trusted signatures!
# gpg:          It is not certain that the signature belongs to the owner.
# Primary key fingerprint: 3635 C788 CE62 B91F D4C5  9AB4 7B02 44FB 12DE 4422

* remotes/quic/tags/pull-hex-20211006:
  target/hexagon: Use tcg_constant_*
  target/hexagon: Remove unused TCG temporary from predicated loads
  Hexagon (target/hexagon) probe the stores in a packet at start of commit

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Compare: https://github.com/qemu/qemu/compare/6723ff639cac...ca61fa4b803e