Re: [PATCH] Add -fopt-builtin optimization option

2021-11-02 Thread Keith Packard via Gcc
Richard Biener writes: > I don't think it reliably works the way you implement it. It's also having > more side-effects than what you document, in particular Yeah, I made a 'minimal' patch that had the effect I needed, but it's clearly in the wrong place as it disables the matching of builtins

Re: [PATCH] Add -fopt-builtin optimization option

2021-11-02 Thread Keith Packard via Gcc-patches
Richard Biener writes: > I don't think it reliably works the way you implement it. It's also having > more side-effects than what you document, in particular Yeah, I made a 'minimal' patch that had the effect I needed, but it's clearly in the wrong place as it disables the matching of builtins

Re: [PATCH] Add -fopt-builtin optimization option

2021-11-02 Thread Richard Biener via Gcc
On Sun, Oct 31, 2021 at 11:13 AM Keith Packard via Gcc-patches wrote: > > This option (enabled by default) controls optimizations which convert > a sequence of operations into an equivalent sequence that includes > calls to builtin functions. Typical cases here are code which matches > memcpy,

Re: [PATCH] Add -fopt-builtin optimization option

2021-11-02 Thread Richard Biener via Gcc-patches
On Sun, Oct 31, 2021 at 11:13 AM Keith Packard via Gcc-patches wrote: > > This option (enabled by default) controls optimizations which convert > a sequence of operations into an equivalent sequence that includes > calls to builtin functions. Typical cases here are code which matches > memcpy,

[PATCH] Add -fopt-builtin optimization option

2021-10-31 Thread Keith Packard via Gcc
This option (enabled by default) controls optimizations which convert a sequence of operations into an equivalent sequence that includes calls to builtin functions. Typical cases here are code which matches memcpy, calloc, sincos. The -ftree-loop-distribute-patterns flag only covers converting

[PATCH] Add -fopt-builtin optimization option

2021-10-31 Thread Keith Packard via Gcc-patches
This option (enabled by default) controls optimizations which convert a sequence of operations into an equivalent sequence that includes calls to builtin functions. Typical cases here are code which matches memcpy, calloc, sincos. The -ftree-loop-distribute-patterns flag only covers converting