Re: [PATCH] rs6000: Built-in cleanups for vec_clzm, vec_ctzm, and vec_gnb.

2020-05-11 Thread Segher Boessenkool
On Mon, May 11, 2020 at 09:31:41PM -0500, Bill Schmidt wrote: > On 5/11/20 7:16 AM, Segher Boessenkool wrote: > >>* config/rs6000/rs6000-c.c (altivec_resolve_overloaded_builtin): > >>Change fourth operand for vec_ternarylogic to require > >>compatibility with unsigned SImode rather than

Re: [PATCH] rs6000: Built-in cleanups for vec_clzm, vec_ctzm, and vec_gnb.

2020-05-11 Thread Bill Schmidt via Gcc-patches
On 5/11/20 7:16 AM, Segher Boessenkool wrote: Hi! On Sat, May 09, 2020 at 08:08:34PM -0500, Bill Schmidt wrote: I should have noticed this patch before submitting Kelvin's earlier related patches, sorry. I think it should still be fine to apply the patches in order, but if you'd like me to c

Re: [PATCH] rs6000: Built-in cleanups for vec_clzm, vec_ctzm, and vec_gnb.

2020-05-11 Thread Segher Boessenkool
Hi! On Sat, May 09, 2020 at 08:08:34PM -0500, Bill Schmidt wrote: > I should have noticed this patch before submitting Kelvin's earlier > related patches, sorry. I think it should still be fine to apply > the patches in order, but if you'd like me to combine this into the > two earlier ones, I'd

Re: [PATCH] rs6000: Built-in cleanups for vec_clzm, vec_ctzm, and vec_gnb.

2020-05-09 Thread David Edelsohn via Gcc-patches
On Sat, May 9, 2020 at 9:08 PM Bill Schmidt wrote: > > From: Kelvin Nilsen > > Changes to the built-in specification occurred after early patches > added support for these. The name of vec_clzm became vec_cntlzm, > and vec_ctzm became vec_cnttzm. Four of the overloaded forms of > vec_gnb were r

[PATCH] rs6000: Built-in cleanups for vec_clzm, vec_ctzm, and vec_gnb.

2020-05-09 Thread Bill Schmidt via Gcc-patches
From: Kelvin Nilsen Changes to the built-in specification occurred after early patches added support for these. The name of vec_clzm became vec_cntlzm, and vec_ctzm became vec_cnttzm. Four of the overloaded forms of vec_gnb were removed, and the fourth argument redefined as an unsigned int, not