This is the mail archive of the mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PING]: Use MPFR for Bessel functions j0, j1 and jn

On Tue, 8 May 2007, Richard Guenther wrote:

> On 5/8/07, Kaveh R. GHAZI <> wrote:
> >
> > PING: Use MPFR for Bessel functions j0, j1 and jn
> >
> >
> >
> > Didn't put it in the patch queue the first time around.
> > :ADDPATCH middle-end:
> I don't like this conditionalizing very much.  I'd rather require us that
> mpfr version once it is available.


This requirement puts me in an impossible position.  If you insist on it
you're effectively blocking all of my recent patches because even when
mpfr-2.3.0 is released, there is opposition to requiring later MPFR
versions to bootstrap GCC.  Surely you're aware of this sentiment among
the community, I tried updating the MPFR version a micro level to 2.2.1
and you were one of the first to object:

Other people felt the same way:

Some people objected to the timing, others object to the upgrade under any
circumstances.  I can see the reasoning behind not requiring an upgrade,
we don't want to make it too hard to bootstrap C.  It seems to be the will
of the majority for now, so that's where we stand.

However in using the MPFR version conditionals, it allows me to clear my
queue of patches, and the new functionality is there to get wider testing.
So it's ready to roll once the mpfr-2.3.0 release is cut and more people
start to use it in conjunction with gcc-4.3.  Note: before we required
mpfr-2.2.0, the fortran frontend made heavy use of conditionalizing like
this, so there is precedent for this when we decide to support older MPFR
versions but take advantage of new ones as they become released:

> Though I also don't see that much value in evaluating these functions at
> compile-time ;)
> Richard.

Well you probably don't use bessel functions in your daily routine.
That's not a reason to oppose it.  I've spent a lot of time working with
the MPFR authors to get new features into MPFR 2.3.0 that would be useful
for evaluating the remaining C99 math functions from GCC.  You're not
being respectful of my time or theirs with your dismissal of our efforts.

If there is a way to bridge our differences I would be willing to make
modifications to my patch to address your concerns while still getting the
stuff installed now.

For example, would it help if I xfailed the testcase until mpfr-2.3.0 is
released?  That kind of change makes sense to me in hindsight.  And that's
the kind of constructive feedback I would seek from a maintainer like

Would you please reconsider and offer some path forward?

Kaveh R. Ghazi

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]