From mboxrd@z Thu Jan 1 00:00:00 1970 From: Josh Triplett Subject: Re: [RFC/PATCH 1/2] sparse: Fix including glibc headers on x86-64 Date: Fri, 23 Mar 2012 11:03:20 -0700 Message-ID: <20120323180320.GA17003@leaf> References: <1319373420-8967-1-git-send-email-penberg@cs.helsinki.fi> <1332493309.2882.6.camel@pasglop> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from relay3-d.mail.gandi.net ([217.70.183.195]:47110 "EHLO relay3-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755677Ab2CWSDb (ORCPT ); Fri, 23 Mar 2012 14:03:31 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-sparse-owner@vger.kernel.org List-Id: linux-sparse@vger.kernel.org To: Christopher Li Cc: Benjamin Herrenschmidt , penberg@cs.helsinki.fi, linux-sparse@vger.kernel.org, Pekka Enberg , Jeff Garzik , Linus Torvalds , Frederic Crozat On Fri, Mar 23, 2012 at 03:03:51AM -0700, Christopher Li wrote: > --- a/lib.c > +++ b/lib.c > @@ -224,6 +224,15 @@ static enum { STANDARD_C89, > STANDARD_GNU89, > STANDARD_GNU99, } standard = STANDARD_GNU89; > > +#ifdef __x86_64__ > +#define ARCH_M64_DEFAULT 1 > +#else > +#define ARCH_M64_DEFAULT 0 > +#endif Ideally this should work on other 64-bit architectures as well. Other than that, the rest of the patch looks reasonable. - Josh Triplett