[PATCH] D100476: [AArch64][SVEIntrinsicOpts] Replace last{a,b} intrinsic calls with extracts...

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:292-296
+    Value *Pg = II.getArgOperand(0);
+    Value *Vec = II.getArgOperand(1);
+    bool IsAfter = IID == Intrinsic::aarch64_sve_lasta;
+
+    auto *C = dyn_cast<Constant>(Pg);
----------------
Can the actual combine live in its own function? (for example into instCombineSVELast ) Otherwise this function is going to become unreadable?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100476