[PATCH] D129066: [AArch64][CodeGen] Add AArch64 support for complex deinterleaving

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:23286
+
+  if (TyWidth > 128) {
+    int Stride = Ty->getNumElements() / 2;
----------------
It isn't clear to me why `TyWidth` can have any value greater than 128. Don't we expect it to be a multiple of something?


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:23300
+
+    if (Accumulator) {
+      LowerSplitAcc = B.CreateShuffleVector(Accumulator, LowerSplitMask);
----------------
`Accumulator` always has a value at this point, don't need to check it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129066