[PATCH] D92824: [SLP] Use the width of value truncated just before storing

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 07:57:31 PST 2020


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM - cheers



================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:5547
+      return DL->getTypeSizeInBits(Store->getValueOperand()->getType());
+  }
 
----------------
anton-afanasyev wrote:
> RKSimon wrote:
> > Does InstrElementSize have to updated at all?
> Do you mean to remove its using at all? This is just a cache for sizes needed not to traverse the expression tree multiple times.
Ah - thanks for confirming.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92824



More information about the llvm-commits mailing list