From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45088) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WSkH1-00068g-3h for qemu-devel@nongnu.org; Wed, 26 Mar 2014 05:37:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WSkGw-0006Lm-Hh for qemu-devel@nongnu.org; Wed, 26 Mar 2014 05:37:27 -0400 Received: from lhrrgout.huawei.com ([194.213.3.17]:12995) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WSkGw-0006LT-A2 for qemu-devel@nongnu.org; Wed, 26 Mar 2014 05:37:22 -0400 Message-ID: <53329FC4.8030409@huawei.com> Date: Wed, 26 Mar 2014 10:37:08 +0100 From: Claudio Fontana MIME-Version: 1.0 References: <1394851732-25692-1-git-send-email-rth@twiddle.net> <1394851732-25692-17-git-send-email-rth@twiddle.net> In-Reply-To: <1394851732-25692-17-git-send-email-rth@twiddle.net> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 16/26] tcg-aarch64: Use tcg_out_call for qemu_ld/st List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Richard Henderson , qemu-devel@nongnu.org Cc: peter.maydell@linaro.org, claudio.fontana@gmail.com On 15.03.2014 03:48, Richard Henderson wrote: > In some cases, a direct branch will be in range. > > Signed-off-by: Richard Henderson > --- > tcg/aarch64/tcg-target.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/tcg/aarch64/tcg-target.c b/tcg/aarch64/tcg-target.c > index 34eee6a..3b8aa7d 100644 > --- a/tcg/aarch64/tcg-target.c > +++ b/tcg/aarch64/tcg-target.c > @@ -1072,8 +1072,7 @@ static void tcg_out_qemu_ld_slow_path(TCGContext *s, TCGLabelQemuLdst *lb) > tcg_out_movr(s, TARGET_LONG_BITS == 64, TCG_REG_X1, lb->addrlo_reg); > tcg_out_movi(s, TCG_TYPE_I32, TCG_REG_X2, lb->mem_index); > tcg_out_movi(s, TCG_TYPE_PTR, TCG_REG_X3, (intptr_t)lb->raddr); > - tcg_out_movi(s, TCG_TYPE_I64, TCG_REG_TMP, (intptr_t)qemu_ld_helpers[size]); > - tcg_out_callr(s, TCG_REG_TMP); > + tcg_out_call(s, (intptr_t)qemu_ld_helpers[size]); > if (opc & MO_SIGN) { > tcg_out_sxt(s, TCG_TYPE_I64, size, lb->datalo_reg, TCG_REG_X0); > } else { > @@ -1094,8 +1093,7 @@ static void tcg_out_qemu_st_slow_path(TCGContext *s, TCGLabelQemuLdst *lb) > tcg_out_movr(s, size == MO_64, TCG_REG_X2, lb->datalo_reg); > tcg_out_movi(s, TCG_TYPE_I32, TCG_REG_X3, lb->mem_index); > tcg_out_movi(s, TCG_TYPE_PTR, TCG_REG_X4, (intptr_t)lb->raddr); > - tcg_out_movi(s, TCG_TYPE_I64, TCG_REG_TMP, (intptr_t)qemu_st_helpers[size]); > - tcg_out_callr(s, TCG_REG_TMP); > + tcg_out_call(s, (intptr_t)qemu_st_helpers[size]); > tcg_out_goto(s, (intptr_t)lb->raddr); > } > > Reviewed-by: Claudio Fontana