Christophe Lyon <christophe.l...@linaro.org> writes:
> On 8 October 2015 at 18:55, Richard Sandiford <richard.sandif...@arm.com>
> wrote:
>> Marc Glisse <marc.gli...@inria.fr> writes:
>>> On Mon, 5 Oct 2015, Richard Sandiford wrote:
>>>
>>>> + /* cbrt(sqrt(x)) -> pow(x,1/6). */
>>>> + (simplify
>>>> + (sqrts (cbrts @0))
>>>> + (pows @0 { build_real_truncate (type, dconst<1, 6> ()); }))
>>>> + /* sqrt(cbrt(x)) -> pow(x,1/6). */
>>>> + (simplify
>>>> + (cbrts (sqrts @0))
>>>> + (pows @0 { build_real_truncate (type, dconst<1, 6> ()); }))
>>>
>>> I think you swapped the comments (not that it matters).
>>
>> Thanks, fixed in the committed version.
>>
>> Richard
>>
> Hi Richard,
>
> Since you committed this patch, I've noticed that gcc.dg/builtins-10.c fails
> on arm-none-linux-gnueabi targets (as opposed to arm-none-linux-gnueabihf).
>
> gcc.log shows:
> /cchfHDHc.o: In function `test':
> builtins-10.c:(.text+0x60): undefined reference to `link_error'
> collect2: error: ld returned 1 exit status
Looks like this is the same fold_strip_sign_ops problem that I was seeing
with some WIP follow-on patches. We don't fold pow(abs(x), 4) to pow(x, 4).
Hope to have a fix soon.
Thanks,
Richard
