develooper Front page | perl.perl5.porters | Postings from July 2010

RE: [PATCH] Configure probe for static inline

Thread Previous
From:
Andy Dougherty
Date:
July 8, 2010 09:57
Subject:
RE: [PATCH] Configure probe for static inline
Message ID:
alpine.DEB.2.00.1007081254330.4659@fractal.phys.lafayette.edu
On Thu, 8 Jul 2010, Jan Dubois wrote:

> On Thu, 08 Jul 2010, Andy Dougherty wrote:
> > On Thu, 8 Jul 2010, Marvin Humphrey wrote:
> > 
> > > I'm very happy to see this patch, and I hope the experiment works out.
> > >
> > > It looks like only the variant "inline" is probed for.  I have found it useful
> > > to probe for "__inline" and "__inline__" first and prefer those.  GCC disables
> > > the "inline" keyword under "-ansi":
> > >
> > >     http://gcc.gnu.org/onlinedocs/gcc/Alternate-Keywords.html
> > >
> > > Also, for MSVC, IIRC only "__inline" works.
> > 
> > Thanks for the feedback (and the later verification that MSVC only
> > documents __inline as supported).
> 
> Note though that on older compiler versions it has only a single leading
> underscore: '_inline', so we need to be able to #define it to different
> values.
> 
> But given that Perl doesn't use Configure on Windows I thought it wouldn't
> matter either way.

Thanks.  Knowing that it will need to be redefined is useful -- I'm 
restructuring the patch so that such redefinition is easier to do.  (This 
will also make it easier to catch the corner case of gcc -ansi.)

-- 
    Andy Dougherty		doughera@lafayette.edu

Thread Previous


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About