[PATCH] D80738: [llvm][SVE] IR intrinsic for LD1RO.

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 28 10:24:01 PDT 2020


sdesmalen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:11898
 
+static SDValue performLD1ROCombine(SDNode *N, SelectionDAG &DAG) {
+  SDLoc DL(N);
----------------
fpetrogalli wrote:
> Note: I am tempted to merge `performLD1ROCombine` and `performLD1RQCombine` in a single function, as they only differ in the ISD node being used to replace the intrinsic. Let me know if this makes sense, I'll update the patch.
Yes, that makes sense.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80738





More information about the llvm-commits mailing list