[BUILDROBOT] ./insn-flags.h:342:7: error: ‘operands’ was not declared in this scope (was: Add an rsqrt_optab and IFN_RSQRT internal function)

2015-12-06 Thread Jan-Benedict Glaw
On Thu, 2015-12-03 09:21:03 +, Richard Sandiford wrote: > All current uses of builtin_reciprocal convert 1.0/sqrt into rsqrt. > This patch adds an rsqrt optab and associated internal function for > that instead. We can then pick up the vector forms of rsqrt automatically, > fixing an AArch64

Re: Add an rsqrt_optab and IFN_RSQRT internal function

2015-12-03 Thread Richard Biener
On Thu, Dec 3, 2015 at 10:39 AM, Jakub Jelinek wrote: > On Thu, Dec 03, 2015 at 09:21:03AM +, Richard Sandiford wrote: >> * internal-fn.def (RSQRT): New function. >> * optabs.def (rsqrt_optab): New optab. >> * doc/tm.texi (rsqrtM2): Document > > Missing full stop. > > Otherwi

Re: Add an rsqrt_optab and IFN_RSQRT internal function

2015-12-03 Thread Jakub Jelinek
On Thu, Dec 03, 2015 at 09:21:03AM +, Richard Sandiford wrote: > * internal-fn.def (RSQRT): New function. > * optabs.def (rsqrt_optab): New optab. > * doc/tm.texi (rsqrtM2): Document Missing full stop. Otherwise looks to me like a nice cleanup and hopefully fixes the aarch64

Add an rsqrt_optab and IFN_RSQRT internal function

2015-12-03 Thread Richard Sandiford
All current uses of builtin_reciprocal convert 1.0/sqrt into rsqrt. This patch adds an rsqrt optab and associated internal function for that instead. We can then pick up the vector forms of rsqrt automatically, fixing an AArch64 regression from my internal_fn patches. With that change, builtin_re