Re: Move sqrt and cbrt simplifications to match.pd

[Christophe Lyon <christophe.lyon@linaro.org>]

On 8 October 2015 at 18:55, Richard Sandiford <richard.sandiford@arm.com> wrote:
> Marc Glisse <marc.glisse@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

Does this ring a bell?

Christophe.