[PATCH] D77538: [Alignment][NFC] Assume AlignmentFromAssumptions::getNewAlignment is always set.
Clement Courbet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 6 06:27:41 PDT 2020
courbet added inline comments.
================
Comment at: llvm/lib/Transforms/Scalar/AlignmentFromAssumptions.cpp:181
+ }
+ assert(isPowerOf2_64(NewAlignment) && "valid alignment");
+ assert(isPowerOf2_64(NewIncAlignment) && "valid alignment");
----------------
And now `getNewAlignmentDiff()` can return MaybeAlign, so we do not need that assert.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77538/new/
https://reviews.llvm.org/D77538
More information about the llvm-commits
mailing list