[PATCH] D139810: [AArch64] Add Neon int instructions to isAssociativeAndCommutative

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 16 08:58:10 PST 2022


dmgreen accepted this revision.
dmgreen added reviewers: SjoerdMeijer, labrinea.
dmgreen added a comment.
This revision is now accepted and ready to land.

Sounds like a good set of instructions. LGTM



================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:4996-4998
+  // Opcodes MULv1i64 and MULv2i64 don't exist because corresponding
+  // `MUL <Vd>.1D, <Vn>.1D, <Vm>.1D` and `MUL <Vd>.2D, <Vn>.2D, <Vm>.2D`
+  // don't exist in the Advanced SIMD instruction set.
----------------
This could just say something like "there is no 64bit mul under neon".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139810



More information about the llvm-commits mailing list