[PATCH] D78443: [Local] Update getOrEnforceKnownAlignment/getKnownAlignment to use Align/MaybeAlign.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 18 22:37:45 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:1019
+    Align NewAlign = getOrEnforceKnownAlignment(
+        S0->getPointerOperand(), MaybeAlign(StackAdjustedAlignment), DL, S0,
+        nullptr, &DT);
----------------
Please use Align(), not MaybeAlign(), for values known to be non-zero.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78443





More information about the llvm-commits mailing list