From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25407 invoked by alias); 14 Apr 2006 20:36:16 -0000 Received: (qmail 25398 invoked by uid 22791); 14 Apr 2006 20:36:16 -0000 X-Spam-Check-By: sourceware.org Received: from nevyn.them.org (HELO nevyn.them.org) (66.93.172.17) by sourceware.org (qpsmtpd/0.31.1) with ESMTP; Fri, 14 Apr 2006 20:36:15 +0000 Received: from drow by nevyn.them.org with local (Exim 4.54) id 1FUV1Z-0006Dj-4a; Fri, 14 Apr 2006 16:36:13 -0400 Date: Fri, 14 Apr 2006 20:36:00 -0000 From: Daniel Jacobowitz To: DJ Delorie , gdb-patches@sourceware.org Subject: Re: [v850 sim] correcting psw flags for divide special cases. Message-ID: <20060414203613.GA23889@nevyn.them.org> Mail-Followup-To: DJ Delorie , gdb-patches@sourceware.org References: <200604132336.k3DNan1h001282@greed.delorie.com> <20060414203520.GA23815@nevyn.them.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060414203520.GA23815@nevyn.them.org> User-Agent: Mutt/1.5.8i X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-04/txt/msg00198.txt.bz2 On Fri, Apr 14, 2006 at 04:35:20PM -0400, Daniel Jacobowitz wrote: > On Thu, Apr 13, 2006 at 07:36:49PM -0400, DJ Delorie wrote: > > > > 2006-04-13 DJ Delorie > > > > * simops.c (OP_2C207E0): Correct PSW flags for special divu > > conditions. > > (OP_2C007E0): Likewise, for div. > > (OP_28207E0): Likewise, for divhu. > > (OP_28007E0): Likewise, for divh. > > Hi DJ, > > The problem with patches without any human-readable text is we don't > know what you expect of them... are you asking for approval? If so, > I'm not sure why, since you're the m32c sim maintainer and few others > on this list even know what an m32c is. If you're just posting the > patch as an "FYI, I committed this", then I'd like to point out that > you didn't commit it :-) Er, this was a v850 patch, not an m32c patch. D'oh. Is there anyone out there who can review v850 changes? If not, I don't see any reason why you shouldn't check them in. -- Daniel Jacobowitz CodeSourcery