Re: [PATCH] i386: Fix up *minmax3_2 splitter [PR116925]

2024-10-04 Thread Uros Bizjak
On Fri, Oct 4, 2024 at 12:07 PM Jakub Jelinek wrote: > > Hi! > > While *minmax3_1 correctly uses >if (MEM_P (operands[1])) > operands[1] = force_reg (mode, operands[1]); > to ensure operands[1] is not a MEM, *minmax3_2 does it wrongly > by calling force_reg but ignoring its return value.

[PATCH] i386: Fix up *minmax3_2 splitter [PR116925]

2024-10-04 Thread Jakub Jelinek
Hi! While *minmax3_1 correctly uses if (MEM_P (operands[1])) operands[1] = force_reg (mode, operands[1]); to ensure operands[1] is not a MEM, *minmax3_2 does it wrongly by calling force_reg but ignoring its return value. The following borderingly obvious patch fixes that, bootstrapped/reg