[PATCH] D120570: [SVE] Add pattern to commute FMSB operands
Paul Walker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 28 13:31:06 PST 2022
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
----------------
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.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D120570/new/
https://reviews.llvm.org/D120570
More information about the llvm-commits
mailing list