[prev in list] [next in list] [prev in thread] [next in thread] 

List:       perl5-porters
Subject:    Re: [perl #131689] Configure should probe for __builtin_add_overflow
From:       "H.Merijn Brand" <h.m.brand () xs4all ! nl>
Date:       2017-10-29 8:36:31
Message-ID: 20171029093557.3e6f17c8 () pc09 ! procura ! nl
[Download RAW message or body]


On Sat, 28 Oct 2017 13:46:58 -0700, "James E Keenan via RT"
<perlbug-followup@perl.org> wrote:

> On Sun, 02 Jul 2017 08:19:31 GMT, mauke- wrote:
> > On Sun, 02 Jul 2017 01:14:12 -0700, mauke- wrote:  
> > >
> > > I'd like to be able __builtin_add_overflow, __builtin_sub_overflow,
> > > __builtin_mul_overflow (safe +, -, * with overflow checks) in core
> > > code.
> > >
> > > It would be nice if Configure had checks for those (it already checks
> > > for
> > > __builtin_choose_expr and __builtin_expect).  
> > 
> > I've attached patches to do that, but I'm not sure if this is the
> > right way to do it (see bug #131688).
> > 
> > The first patch extends Configure.
> > 
> > The second patch extends config_h.SH, which contains a comment saying
> > "THIS IS A GENERATED FILE. DO NOT HAND-EDIT.  See
> > Porting/config_h.pl", but I saw Porting/config_h.pl and it contains no
> > useful information, so I hand-edited anyway.  
> 
> Now that metaconfig has its repository on github.com, should this RT
> be re-formulated as an issue at that location?

As these have already been merged into Configure, this ticket should
simply be closed.

> https://github.com/perl5-metaconfig/metaconfig/issues?q=is%3Aopen+is%3Aissue

-- 
H.Merijn Brand  http://tux.nl   Perl Monger  http://amsterdam.pm.org/
using perl5.00307 .. 5.27   porting perl5 on HP-UX, AIX, and openSUSE
http://mirrors.develooper.com/hpux/        http://www.test-smoke.org/
http://qa.perl.org   http://www.goldmark.org/jeff/stupid-disclaimers/

[Attachment #3 (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]