[PATCH] D103419: [VectorCombine] Fix alignment in single element store

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 31 10:44:16 PDT 2021


tschuett added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VectorCombine.cpp:834
     NSI->copyMetadata(*SI);
-    if (SI->getAlign() < NSI->getAlign())
-      NSI->setAlignment(SI->getAlign());
+    Align NewAlignment(1);
+    if (auto *C = dyn_cast<ConstantInt>(Idx)) {
----------------
This line is too cute for me, but ...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103419



More information about the llvm-commits mailing list