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

Re: [PATCH] Configure probe for static inline

Thread Previous | Thread Next
From:
H.Merijn Brand
Date:
July 9, 2010 04:04
Subject:
Re: [PATCH] Configure probe for static inline
Message ID:
20100709130406.26c928c5@pc09.procura.nl
On Fri, 9 Jul 2010 11:31:15 +0100, Nicholas Clark <nick@ccl4.org> wrote:

> On Thu, Jul 08, 2010 at 02:14:34PM +0200, H.Merijn Brand wrote:
> > On Tue, 6 Jul 2010 12:48:56 -0400 (EDT), Andy Dougherty
> > <doughera@lafayette.edu> wrote:
> > 
> > > Here is a proposed patch to enable Configure to probe for C99-style 
> > > 'static inline'.  (That is, functions may be inlined, but will not be 
> > > externally visible.)  The initial idea is that some common code in messy 
> > > macros inside headers might be simplified using inline functions.  If the 
> > > compiler does not support 'static inline', then a plain 'static' is used 
> > > instead, along with the consequent implications of a function call.  In 
> > > either case, you simply use  PERL_STATIC_INLINE.
> 
> Thanks for working on this.
> This will enable a lot of cleanup of existing messy macros.
> 
> > I suggest just committing this and see how smokes react
> 
> I'd suggest this too. *I* don't know how to take a metaconfig patch and use
> it to rebuild Configure and config_h.SH, so I'm doubtful that many other
> people will either.

Backporting isn't that hard. Done that, been there.

> I suspect it will only get realistic testing if it's in blead, and smokers
> (and humans) try it as a side effect of building blead for other reasons.

Correct.

> However, I'm not sure how many people have access to non-gcc, and particularly
> non-gcc non x86 (or x86_64) platforms, to see how those compilers' code
> generators react to changing from macros to inline functions. So it's going
> to be tricky to really get good confidence on what the impact of changes are,
> outside VAX 2.0*

My AIX and HP-UX smokes are cron-based. Unless weird things happen
during my absence, they keer running. You know where to find the (full)
reports.

> Nicholas Clark
> 
> *  The thing that all the world is. [x86, gcc, ELF, ld.so, these days.**]
> ** Which may change, if portable devices usurp laptops, and battery life
>    favours ARMs.

-- 
H.Merijn Brand  http://tux.nl      Perl Monger  http://amsterdam.pm.org/
using 5.00307 through 5.12 and porting perl5.13.x on HP-UX 10.20, 11.00,
11.11, 11.23, and 11.31, OpenSuSE 10.3, 11.0, and 11.1, AIX 5.2 and 5.3.
http://mirrors.develooper.com/hpux/           http://www.test-smoke.org/
http://qa.perl.org      http://www.goldmark.org/jeff/stupid-disclaimers/

Thread Previous | Thread Next


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