[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