[PATCH] D77538: [Alignment][NFC] Assume AlignmentFromAssumptions::getNewAlignment is always set.

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 6 05:55:18 PDT 2020


gchatelet marked an inline comment as done.
gchatelet added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/AlignmentFromAssumptions.cpp:182
-    } else if (NewAlignment > NewIncAlignment) {
-      if (NewAlignment % NewIncAlignment == 0) {
-        LLVM_DEBUG(dbgs() << "\tnew start/inc alignment: " << NewIncAlignment
----------------
I could remove these checks since we know both NewAlignment and NewIncAlignment are powers of two (and so are multiples of each other)


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