[PATCH] D71074: [Aarch64][SVE] Add intrinsics for scatter stores

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 11 07:31:27 PST 2019


sdesmalen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:11861
+  EVT MaxVT = llvm::MVT::getScalableVectorVT(SrcElVT, NumElements);
+  if (SrcVT.getSizeInBits().getKnownMinSize() >
+      MaxVT.getSizeInBits().getKnownMinSize())
----------------
sdesmalen wrote:
> Should we simply compare:
> ```SrcVT.getSizeInBits().getScalableSize() > AArch64::SVEBitsPerBlock```
> ?
Sorry, I meant:
```SrcVT.getSizeInBits().getKnownMinSize() > AArch64::SVEBitsPerBlock```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71074





More information about the llvm-commits mailing list