Re: [PATCH 11/18] rs6000: Builtin expansion, part 6

2021-11-08 Thread Bill Schmidt via Gcc-patches
Sorry for the misunderstanding.  What I meant is the 6 patches entitled "Builtin expansion, part N". I still have 6-7 patches left to look at. Thanks! Bill On 11/7/21 3:05 PM, Segher Boessenkool wrote: > Hi! > > On Sun, Nov 07, 2021 at 09:28:09AM -0600, Bill Schmidt wrote: >> Thank you for all

Re: [PATCH 11/18] rs6000: Builtin expansion, part 6

2021-11-07 Thread Segher Boessenkool
Hi! On Sun, Nov 07, 2021 at 09:28:09AM -0600, Bill Schmidt wrote: > Thank you for all of the reviews!  I appreciate your hard work and thorough > study of the patches. > > I've updated these 6 patches and combined them into 1, pushed today.  There > are still a couple of cleanups I haven't

Re: [PATCH 11/18] rs6000: Builtin expansion, part 6

2021-11-07 Thread Bill Schmidt via Gcc-patches
Hi Segher, Thank you for all of the reviews!  I appreciate your hard work and thorough study of the patches. I've updated these 6 patches and combined them into 1, pushed today.  There are still a couple of cleanups I haven't done, but I made note in the code where these are needed. Thanks

Re: [PATCH 11/18] rs6000: Builtin expansion, part 6

2021-11-03 Thread Segher Boessenkool
Hi! On Wed, Sep 01, 2021 at 11:13:47AM -0500, Bill Schmidt wrote: > Provide replacements for htm_spr_num and htm_expand_builtin. No logic > changes are intended here, as usual. Much code was factored out into > rs6000_expand_new_builtin, so the new version of htm_expand_builtin is > a little

[PATCH 11/18] rs6000: Builtin expansion, part 6

2021-09-01 Thread Bill Schmidt via Gcc-patches
Provide replacements for htm_spr_num and htm_expand_builtin. No logic changes are intended here, as usual. Much code was factored out into rs6000_expand_new_builtin, so the new version of htm_expand_builtin is a little tidier. Also implement the support for the "endian" and "32bit" attributes,