[PATCH] D120912: [AArch64][SVE] Convert gather/scatter with a stride of 2 to contiguous loads/stores

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 11 06:04:30 PST 2022


david-arm added a comment.

LGTM (with formatting issues fixed)! This looks great now thanks @kmclaughlin. :)



================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:16773
 
+static SDValue tryToCombineToMaskedLoadStore(MaskedGatherScatterSDNode *MGS,
+                                             SelectionDAG &DAG,
----------------
nit: Can you fix all the formatting issues in your new code before merging please?


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

https://reviews.llvm.org/D120912



More information about the llvm-commits mailing list