This revision was automatically updated to reflect the committed changes.
Closed by commit rL373595: [Alignment][NFC] Remove
StoreInst::setAlignment(unsigned) (authored by gchatelet, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D68268?vs=222991&id=223000#toc
Repository:
gchatelet added inline comments.
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:4027
- ST->setAlignment(Alignment);
+ ST->setAlignment(Align(Alignment));
Value *V = propagateMetadata(ST, E->Scalars);
courbet wrote:
> I don't think
gchatelet updated this revision to Diff 222991.
gchatelet marked 6 inline comments as done.
gchatelet added a comment.
- Address comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68268/new/
https://reviews.llvm.org/D68268
Files:
clang/lib/C
courbet added inline comments.
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:189
+ L->setAlignment(
+ MaybeAlign(CopySrcAlign)); // Check if we can use Align instead.
if (CopyMD)
FIXME
Comment at: llvm/lib/Transforms
gchatelet created this revision.
gchatelet added a reviewer: courbet.
Herald added a reviewer: bollu.
Herald added subscribers: llvm-commits, cfe-commits, asbirlea, hiraditya.
Herald added a reviewer: jdoerfert.
Herald added projects: clang, LLVM.
This is patch is part of a series to introduce an
