[PATCH] D120570: [SVE] Add pattern to commute FMSB operands

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/sve-fp.ll:321-323
+  %neg = fneg <vscale x 8 x half> %b
+  %r = call <vscale x 8 x half> @llvm.fma.nxv8f16(<vscale x 8 x half> %a, <vscale x 8 x half> %neg, <vscale x 8 x half> %c)
+  ret <vscale x 8 x half> %r
----------------
paulwalker-arm wrote:
> Can these tests treat the operands the same as the `fma` tests.  You're currently treating `%c` as the addend whereas the existing tests have this as `%a`.  The output will likely change to `fmls` instead of `fmsb` but that seems find to me. 
I've just checked and these negated patterns are tested within `sve-fp-combine.ll` so I think that'll be a better home for your new tests.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120570/new/

https://reviews.llvm.org/D120570