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 Does this ring a bell? Christophe.