From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:45361) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QFPRN-0003CC-F9 for qemu-devel@nongnu.org; Thu, 28 Apr 2011 07:31:26 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QFPRM-0005ik-8J for qemu-devel@nongnu.org; Thu, 28 Apr 2011 07:31:25 -0400 Received: from mail-vx0-f173.google.com ([209.85.220.173]:61900) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QFPRM-0005ie-1X for qemu-devel@nongnu.org; Thu, 28 Apr 2011 07:31:24 -0400 Received: by vxb37 with SMTP id 37so433758vxb.4 for ; Thu, 28 Apr 2011 04:31:22 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <1303311120-3424-1-git-send-email-peter.maydell@linaro.org> References: <1303311120-3424-1-git-send-email-peter.maydell@linaro.org> Date: Thu, 28 Apr 2011 12:31:22 +0100 Message-ID: From: Peter Maydell Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH] target-arm: Move VLD/VST multiple into helper functions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: patches@linaro.org Ping? The two UNDEF patches have both been applied, so this patch applies cleanly to master now. -- PMM On 20 April 2011 15:52, Peter Maydell wrote: > Move VLD/VST multiple into helper functions, as some cases can > generate more TCG ops than the maximum per-instruction limit > and certainly more than the recommended 20. > > Signed-off-by: Peter Maydell > --- > This patch is inspired by one from the meego tree: > =C2=A0http://git.linaro.org/gitweb?p=3Dqemu/qemu-linaro.git;a=3Dcommitdif= f;h=3Da5b2a79c7929726bac5157783de81d22793efd12 > but I've reworked it to do the decoding at translate time rather > than in the helper function. > > It is intended to apply on top of the neon load/store UNDEF fixes: > http://patchwork.ozlabs.org/patch/91824/ > http://patchwork.ozlabs.org/patch/91825/ > > but I thought it would be better to push it out now for review > rather than waiting for those to be committed. > > I hope you all like macros :-) > > =C2=A0target-arm/helper.h =C2=A0 =C2=A0 =C2=A0| =C2=A0 40 +++++++++++++ > =C2=A0target-arm/neon_helper.c | =C2=A0127 ++++++++++++++++++++++++++++++= +++++++++++ > =C2=A0target-arm/translate.c =C2=A0 | =C2=A0140 +++++++++++--------------= --------------------- > =C2=A03 files changed, 199 insertions(+), 108 deletions(-)