On 14/10/26 20:25, Pierrick Bouvier wrote:
Now that all targets implement this interface, we can replace function
calls with the new interface.

Reviewed-by: Brian Cain <[email protected]>
Signed-off-by: Pierrick Bouvier <[email protected]>
---
  semihosting/arm-compat-semi.c | 56 +++++++++++++++++------------------
  1 file changed, 28 insertions(+), 28 deletions(-)

diff --git a/semihosting/arm-compat-semi.c b/semihosting/arm-compat-semi.c
index b930a028649..3117eadb3b7 100644
--- a/semihosting/arm-compat-semi.c
+++ b/semihosting/arm-compat-semi.c
@@ -188,7 +188,7 @@ static LayoutInfo common_semi_find_bases(CPUState *cs)
   */
#define GET_ARG(n) do { \
-    if (is_64bit_semihosting(env)) {                    \
+    if (cs->cc->semi_ops->is_64bit(cs)) {               \

No semi_ops / handler NULL deref checks, SEGV as implementation error
sounds OK but might deserve a comment in the commit description.

Reply via email to