From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7058 invoked by alias); 18 Nov 2013 11:38:26 -0000 Mailing-List: contact libffi-discuss-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libffi-discuss-owner@sourceware.org Received: (qmail 7013 invoked by uid 89); 18 Nov 2013 11:38:25 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=0.8 required=5.0 tests=BAYES_20,RDNS_NONE,SPF_HELO_PASS autolearn=no version=3.3.2 X-HELO: mx1.redhat.com Received: from Unknown (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 18 Nov 2013 11:38:24 +0000 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id rAIBcGLB009678 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Mon, 18 Nov 2013 06:38:17 -0500 Received: from localhost.localdomain.redhat.com (vpn-227-142.phx2.redhat.com [10.3.227.142]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id rAIBcFV5031443 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Mon, 18 Nov 2013 06:38:16 -0500 From: Anthony Green To: Alan Modra Subject: Re: Straying #endif in ppc_closure.S In-Reply-To: <20131117031650.GR22514@bubble.grove.modra.org> (Alan Modra's message of "Sun, 17 Nov 2013 13:46:50 +1030") References: <20131117031650.GR22514@bubble.grove.modra.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) CC: libffi-discuss@sourceware.org Date: Mon, 18 Nov 2013 11:38:00 -0000 Message-ID: <87eh6e2b4o.fsf@redhat.com> MIME-Version: 1.0 Content-Type: text/plain X-IsSubscribed: yes X-SW-Source: 2013/txt/msg00214.txt.bz2 Alan Modra writes: > An #endif in the wrong place would cause compile failure on powerpcle. > Using bl instead of b doesn't cause runtime failures as you might think, > but does mess the processor branch prediction. Thanks Alan. I've committed this along with all of your other recent changes. I'm also about to update the README to mention ELFv2 support. AG