From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:51228)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <rth7680@gmail.com>) id 1SB6Mk-0007uj-5m
	for qemu-devel@nongnu.org; Fri, 23 Mar 2012 11:25:25 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <rth7680@gmail.com>) id 1SB6Me-00016N-0u
	for qemu-devel@nongnu.org; Fri, 23 Mar 2012 11:25:21 -0400
Received: from mail-yx0-f173.google.com ([209.85.213.173]:46482)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <rth7680@gmail.com>) id 1SB6Md-00016F-Pz
	for qemu-devel@nongnu.org; Fri, 23 Mar 2012 11:25:15 -0400
Received: by yenr5 with SMTP id r5so3420137yen.4
	for <qemu-devel@nongnu.org>; Fri, 23 Mar 2012 08:25:14 -0700 (PDT)
Sender: Richard Henderson <rth7680@gmail.com>
Message-ID: <4F6C95D8.50902@twiddle.net>
Date: Fri, 23 Mar 2012 08:25:12 -0700
From: Richard Henderson <rth@twiddle.net>
MIME-Version: 1.0
References: <CAAu8pHuBQ1U9rLuGPuUca3gXv_9erZNrfrj8jezf3ZA4KSAjkg@mail.gmail.com>
In-Reply-To: <CAAu8pHuBQ1U9rLuGPuUca3gXv_9erZNrfrj8jezf3ZA4KSAjkg@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH 6/6] arm: move load and store helpers,
 switch to AREG0 free mode
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Blue Swirl <blauwirbel@gmail.com>
Cc: Peter Maydell <peter.maydell@linaro.org>, Paul Brook <paul@codesourcery.com>, qemu-devel <qemu-devel@nongnu.org>

On 03/19/12 14:57, Blue Swirl wrote:
> Add an explicit CPUARMState parameter instead of relying on AREG0
> and move load and store helpers to helper.c. Remove AREG0 swapping in
> tlb_fill(). Remove now empty op_helper.c.
> 
> Switch to AREG0 free mode. Use cpu_ld{l,uw}_code in translation
> and interrupt handling.
> 
> Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
> ---
>  Makefile.target        |    4 +-
>  configure              |    2 +-
>  target-arm/helper.c    |   54 +++++++++++++++++++++++++++++++++---
>  target-arm/op_helper.c |   71 ------------------------------------------------
>  target-arm/translate.c |    6 ++--
>  5 files changed, 56 insertions(+), 81 deletions(-)
>  delete mode 100644 target-arm/op_helper.c

Patches 2-6 (patch 1 mail lost?):

Reviewed-by: Richard Henderson <rth@twiddle.net>


r~