On 26/06/2025 15:35, Robin Dapp wrote:
This is a followup to 92e1893e0 "RISC-V: Add patterns for vector-scalar
multiply-(subtract-)accumulate" that caused an ICE in some cases where
the mult
operands were wrongly swapped.
This patch ensures that operands are not swapped in the vector-scalar
case.
This looks reasonable, so OK for the trunk but how did that slip through
in the first place?
I guess I missed it when I first ran the testsuite before sending the
patch for review. I rebased and re-ran the testsuite after getting
approved and saw the regression. But at that point I realised Jeff had
already merged it.
Anyway, I'll regtest more carefully next time!
Thanks,
--
PA