Re: [PATCH] Fix DImode to TImode sign extend issue, PR target/104868

2022-03-11 Thread Michael Meissner via Gcc-patches
On Fri, Mar 11, 2022 at 02:41:05PM -0600, Segher Boessenkool wrote: > On Fri, Mar 11, 2022 at 01:07:29AM -0500, Michael Meissner wrote: > > Fix DImode to TImode sign extend issue, PR target/104898 > > > When I wrote the extendditi2 pattern, I forgot that mtvsrdd had that > > behavior so I used a '

Re: [PATCH] Fix DImode to TImode sign extend issue, PR target/104868

2022-03-11 Thread Segher Boessenkool
On Fri, Mar 11, 2022 at 01:07:29AM -0500, Michael Meissner wrote: > Fix DImode to TImode sign extend issue, PR target/104898 > When I wrote the extendditi2 pattern, I forgot that mtvsrdd had that > behavior so I used a 'r' constraint instead of 'b'. In the rare case > where the value is in GPR re

Re: [PATCH] Fix DImode to TImode sign extend issue, PR target/104868

2022-03-11 Thread Michael Meissner via Gcc-patches
Matheus Castanho reports that the patch I posted fixes the problem in the 1040868 bug report. -- Michael Meissner, IBM PO Box 98, Ayer, Massachusetts, USA, 01432 email: meiss...@linux.ibm.com

[PATCH] Fix DImode to TImode sign extend issue, PR target/104868

2022-03-10 Thread Michael Meissner via Gcc-patches
Fix DImode to TImode sign extend issue, PR target/104898 PR target/104868 had had an issue where my code that updated the DImode to TImode sign extension for power10 failed. In looking at the failure message, the reason is when extendditi2 tries to split the insn, it generates an insn that does n