[PATCH] D62386: [AArch64][SVE2] Asm: support Floating Point Widening Multiply-Add

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 24 08:45:53 PDT 2019


c-rhodes added inline comments.


================
Comment at: test/MC/AArch64/SVE/movprfx-diagnostics.s:114
 // ------------------------------------------------------------------------- //
+// Destination used in other operand (ternary + indexed)
+
----------------
chill wrote:
> Stray patch?
This is just testing indexed + movprfxable gives a diagnostic. I understand it appears to be a rogue patch given the instruction is unrelated but the reason for adding it here was we noticed there were no tests for this instruction form in movprfx-diagnostics, initially thinking indexed + movprfxable was new to SVE2 until realising the SVE SDOT instruction is similar, so a test was added for that.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62386





More information about the llvm-commits mailing list