[PATCH] D104217: [AArch64][SVE] Add support for fixed length MSCATTER/MGATHER

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 15 04:57:51 PDT 2021


paulwalker-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:17808-17809
 
+SDValue AArch64TargetLowering::LowerMGATHER(SDValue Op,
+                                            SelectionDAG &DAG) const {
+  SDLoc DL(Op);
----------------
I guess you've moved these lowering functions because of the placement of the fixed length utility functions.  I guess you could move the utility functions but I'm wondering if you can just add forward declarations to the top somewhere?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104217



More information about the llvm-commits mailing list