Re: [PATCH] Fix for PR ipa/64503

[Uros Bizjak <ubizjak@gmail.com>]

On Thu, Jan 8, 2015 at 6:04 PM, Jakub Jelinek <jakub@redhat.com> wrote:
> On Wed, Jan 07, 2015 at 01:43:59PM +0100, Uros Bizjak wrote:
>> Hello!
>>
>> >>>> There's suggested patch for PR ipa/64503 that was tested on x86_64 and it works.
>> >>>> I would like to ask Uros to test it on an aplha machine before we install the patch.
>> >>>
>> >>> Yes, this works for me on all IPA tests that were failing previously [1].
>> >>>
>> >>> I am restarting the bootstrap + regtest, it will take ~10 hours, but I
>> >>> don't expect any surprises there.
>> >>
>> >> The patch was bootstrapped and regression tested on
>> >> alphaev68-linux-gnu [2] and everything was OK.
>> >
>> > Thanks for testing. May I take this as approval of the patch.
>> > I don't know, if you have a permission to accept my patch ;)
>>
>> Unfortunately, I have no approval power here. Luckilly, I know someone
>> who has ;)
>
> I don't know how portable scalbnl actually is in reality, so it would be
> nice to hear from port maintainers whether say newlib, android, darwin,
> cygwin, aix etc. have it.

Newlib has it.

Other than that:

       exp2(), exp2f(), exp2l():
           _XOPEN_SOURCE >= 600 || _ISOC99_SOURCE || _POSIX_C_SOURCE >= 200112L;
           or cc -std=c99

       scalbln(), scalblnf(), scalblnl():
              _XOPEN_SOURCE >= 600 || _ISOC99_SOURCE ||
_POSIX_C_SOURCE >= 200112L;
              or cc -std=c99

Since we are replacing exp2 with scalbln, we have the same
requirements as before, at least we won't regress here.

Uros.