Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-14 Thread Segher Boessenkool
On Fri, Aug 13, 2021 at 07:57:53PM -0400, Michael Meissner wrote: > On Fri, Aug 13, 2021 at 04:33:26PM -0400, David Edelsohn wrote: > > There is a song from Sesame Street: "Which of these is not like the > > others?" altivec.md seems like an outlier. crypto.md and vsx.md also > > seem unusual. >

Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-14 Thread Segher Boessenkool
On Fri, Aug 13, 2021 at 04:33:26PM -0400, David Edelsohn wrote: > On Fri, Aug 13, 2021 at 4:24 PM Segher Boessenkool > wrote: > > On Fri, Aug 13, 2021 at 02:07:25PM -0400, David Edelsohn wrote: > > > On Fri, Aug 13, 2021 at 12:08 PM Segher Boessenkool > > > wrote: > > > > On Fri, Aug 13, 2021 at

Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-13 Thread Michael Meissner via Gcc-patches
On Fri, Aug 13, 2021 at 04:33:26PM -0400, David Edelsohn wrote: > There is a song from Sesame Street: "Which of these is not like the > others?" altivec.md seems like an outlier. crypto.md and vsx.md also > seem unusual. > > We have > > register_operand > gpc_reg_operand > altivec_register_oper

Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-13 Thread David Edelsohn via Gcc-patches
On Fri, Aug 13, 2021 at 4:24 PM Segher Boessenkool wrote: > > On Fri, Aug 13, 2021 at 02:07:25PM -0400, David Edelsohn wrote: > > On Fri, Aug 13, 2021 at 12:08 PM Segher Boessenkool > > wrote: > > > > > > On Fri, Aug 13, 2021 at 11:15:21AM -0400, David Edelsohn wrote: > > > > On Fri, Aug 13, 2021

Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-13 Thread Segher Boessenkool
On Fri, Aug 13, 2021 at 02:07:25PM -0400, David Edelsohn wrote: > On Fri, Aug 13, 2021 at 12:08 PM Segher Boessenkool > wrote: > > > > On Fri, Aug 13, 2021 at 11:15:21AM -0400, David Edelsohn wrote: > > > On Fri, Aug 13, 2021 at 10:49 AM Segher Boessenkool > > > wrote: > > > > > > > > On Fri, Aug

Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-13 Thread David Edelsohn via Gcc-patches
On Fri, Aug 13, 2021 at 12:08 PM Segher Boessenkool wrote: > > On Fri, Aug 13, 2021 at 11:15:21AM -0400, David Edelsohn wrote: > > On Fri, Aug 13, 2021 at 10:49 AM Segher Boessenkool > > wrote: > > > > > > On Fri, Aug 13, 2021 at 12:14:14AM -0400, Michael Meissner wrote: > > > > I noticed that th

Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-13 Thread Segher Boessenkool
On Fri, Aug 13, 2021 at 11:15:21AM -0400, David Edelsohn wrote: > On Fri, Aug 13, 2021 at 10:49 AM Segher Boessenkool > wrote: > > > > On Fri, Aug 13, 2021 at 12:14:14AM -0400, Michael Meissner wrote: > > > I noticed that the xxeval built-in function used the > > > altivec_register_operand > > >

Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-13 Thread David Edelsohn via Gcc-patches
On Fri, Aug 13, 2021 at 10:49 AM Segher Boessenkool wrote: > > On Fri, Aug 13, 2021 at 12:14:14AM -0400, Michael Meissner wrote: > > I noticed that the xxeval built-in function used the > > altivec_register_operand > > predicate. Since it takes vsx registers, this might force the register > > al

Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-13 Thread Segher Boessenkool
On Fri, Aug 13, 2021 at 12:14:14AM -0400, Michael Meissner wrote: > I noticed that the xxeval built-in function used the altivec_register_operand > predicate. Since it takes vsx registers, this might force the register > allocate to issue a move when it could use a traditional floating point > reg

Re: [PATCH] Fix xxeval predicates (PR 99921).

2021-08-13 Thread David Edelsohn via Gcc-patches
On Fri, Aug 13, 2021 at 12:14 AM Michael Meissner wrote: > > Fix xxeval predicates (PR 99921). > > I originally posted this patch in May and in June. I'm reposting it now. > > I noticed that the xxeval built-in function used the altivec_register_operand > predicate. Since it takes vsx registers,

[PATCH] Fix xxeval predicates (PR 99921).

2021-08-12 Thread Michael Meissner via Gcc-patches
Fix xxeval predicates (PR 99921). I originally posted this patch in May and in June. I'm reposting it now. I noticed that the xxeval built-in function used the altivec_register_operand predicate. Since it takes vsx registers, this might force the register allocate to issue a move when it could