From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50581) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XGr8D-0007d6-TA for qemu-devel@nongnu.org; Mon, 11 Aug 2014 11:03:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XGr82-0006Wj-95 for qemu-devel@nongnu.org; Mon, 11 Aug 2014 11:03:29 -0400 Received: from mail.uni-paderborn.de ([131.234.142.9]:13550) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XGr82-0006Vd-3X for qemu-devel@nongnu.org; Mon, 11 Aug 2014 11:03:18 -0400 Message-ID: <53E8EA15.6050905@mail.uni-paderborn.de> Date: Mon, 11 Aug 2014 17:06:45 +0100 From: Bastian Koppelmann MIME-Version: 1.0 References: <1407422081-9468-1-git-send-email-kbastian@mail.uni-paderborn.de> <1407422081-9468-2-git-send-email-kbastian@mail.uni-paderborn.de> <53E435EA.3070408@twiddle.net> In-Reply-To: <53E435EA.3070408@twiddle.net> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4 01/15] target-tricore: Add target stubs and qom-cpu List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Richard Henderson , qemu-devel@nongnu.org Cc: peter.maydell@linaro.org On 08/08/2014 03:28 AM, Richard Henderson wrote: > On 08/07/2014 04:34 AM, Bastian Koppelmann wrote: >> + /* PSW flag cache for faster execution >> + if flag != 0 then flag is set. Else flag is not set. >> + */ >> + target_ulong PSW_USB_C; >> + target_ulong PSW_USB_V; >> + target_ulong PSW_USB_SV; >> + target_ulong PSW_USB_AV; /* Only if bit 31 set, then flag is set. */ >> + target_ulong PSW_USB_SAV; /* Only if bit 31 set, then flag is set. */ > V and SV are also only set if bit 31 is set, the way we're computing overflow > from addition. Of course, overflow from saturation or multiplication isn't > being computed into bit 31, so there is a decision to make. > > Depending on how important it is for ADDX+ADDC to be implemented efficiently, > vs how important is for SHA to be quick, you may wish to have C already set to > 0/1 only. Since I don't have data, which instructions are more common, I would choose the SHA instructions to be fast, because the manual states, that the most commen instructions are also implemented as 16 bit instructions and SHA is part of those. So I would leave the carry bit as is, unless you are sure, that there is a benefit for ADDX+ADDC to be fast. > > > r~