Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Sun, 12 Jan 2003 18:22:52 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Sun, 12 Jan 2003 18:22:52 -0500 Received: from mailout04.sul.t-online.com ([194.25.134.18]:60896 "EHLO mailout04.sul.t-online.com") by vger.kernel.org with ESMTP id convert rfc822-to-8bit; Sun, 12 Jan 2003 18:22:51 -0500 Content-Type: text/plain; charset=US-ASCII From: Oliver Neukum To: robw@optonline.net, Aaron Lehmann Subject: Re: any chance of 2.6.0-test*? Date: Mon, 13 Jan 2003 00:31:24 +0100 User-Agent: KMail/1.4.3 Cc: Rik van Riel , Matti Aarnio , Linux Kernel Mailing List References: <20030112225228.GP31238@vitelus.com> <1042413101.3162.184.camel@RobsPC.RobertWilkens.com> In-Reply-To: <1042413101.3162.184.camel@RobsPC.RobertWilkens.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7BIT Message-Id: <200301130031.24169.oliver@neukum.name> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1121 Lines: 36 Am Montag, 13. Januar 2003 00:11 schrieb Rob Wilkens: > On Sun, 2003-01-12 at 17:52, Aaron Lehmann wrote: > > On Sun, Jan 12, 2003 at 05:34:58PM -0500, Rob Wilkens wrote: > > > You're wrong. You wouldn't have to jump over them any more than you > > > have to jump over the "goto" statement. > > > > The goto is a conditional jump. You propose replacing it with a > > conditional jump past the error handling code predicated on the > > opposite condition. Where's the improvement? > > The goto is absolutely not a conditional jump. The if that precedes it > is conditional. The goto is not. The if is already there. Oh, well. Apologies first, my assembler is rusty. if (a == NULL) goto err_out; bad compiler -> tst $D0 ; evaluate a == NULL bne L1 ; this is the if bra err_out ; this is the goto L1: good compiler -> tst $D0 beq err_out ; obvious optimisation Oliver - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/