Re: [PATCH] rs6000: Refactor altivec_build_resolved_builtin

2021-12-15 Thread Bill Schmidt via Gcc-patches
Hi! On 12/15/21 12:16 PM, Segher Boessenkool wrote: >> + /* Note: vec_nand also works but opt changes vec_nand's >> + to vec_nor's anyway. */ > Maybe there should be a vec_not? There is one at the RTL level (called > one_cmpl2). As I recall, we have an issue open for this

Re: [PATCH] rs6000: Refactor altivec_build_resolved_builtin

2021-12-15 Thread Segher Boessenkool
Hi! On Wed, Dec 15, 2021 at 08:21:24AM -0600, Bill Schmidt wrote: > While replacing the built-in machinery, we agreed to defer some necessary > refactoring of the overload processing. This patch cleans it up considerably. > > I've put in one FIXME for an additional level of cleanup that should

Re: [PATCH] rs6000: Refactor altivec_build_resolved_builtin

2021-12-09 Thread Bill Schmidt via Gcc-patches
I forgot to point out that this patch is dependent on the pending patches to remove the old builtins code. Thanks, Bill On 12/9/21 12:33 PM, Bill Schmidt via Gcc-patches wrote: > Hi! > > While replacing the built-in machinery, we agreed to defer some necessary > refactoring of the overload