[PATCH] D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 3 09:59:46 PDT 2021


arsenm added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:521
     KnownBits Known(BitWidth);
-    computeKnownBits(OpA, Known, DL, 0, &AC, OpA, &DT);
     APInt BitsAllowedToBeSet = Known.Zero.zext(IdxDiff.getBitWidth());
----------------
arsenm wrote:
> How do we know A is the point the vectorized instruction will be inserted, and not B?
e.g. can you add some tests with an assume between the two instructions to vectorize?


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

https://reviews.llvm.org/D101680



More information about the llvm-commits mailing list