[PATCH] D80824: [CodeGen][SVE] Replace deprecated call to getVectorNumElements() with getVectorElemetCount() in replaceZeroVectorStore()

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 3 13:46:24 PDT 2020


sdesmalen added a comment.

After D80720 <https://reviews.llvm.org/D80720> this function will no longer be called from splitStores for scalable vectors, but it still makes sense to bail out early in case that ever changes.



================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:11940
   EVT VT = StVal.getValueType();
 
   // It is beneficial to scalarize a zero splat store for 2 or 3 i64 elements or
----------------
Probably better to bail out directly if VT is scalable and leave the call to `getVectorNumElements()` intact.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80824





More information about the llvm-commits mailing list