[PATCH] D80824: [CodeGen][SVE] Avoid scalarizing zero splat stores on scalable vectors.

Henry Kao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 4 08:12:11 PDT 2020


hkao13 marked an inline comment as done.
hkao13 added inline comments.


================
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
----------------
sdesmalen wrote:
> Probably better to bail out directly if VT is scalable and leave the call to `getVectorNumElements()` intact.
Thanks for the review. I reverted the change to use getVectorNumElements(), and added a check above. 


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

https://reviews.llvm.org/D80824





More information about the llvm-commits mailing list