[PATCH] D69307: [Alignment][NFC] getMemoryOpCost uses MaybeAlign

Clement Courbet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 24 02:32:46 PDT 2019


courbet accepted this revision.
courbet added inline comments.


================
Comment at: llvm/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp:186
+
+    // At this point unspecified alignment is considered as Align::None()..
+    const Align BoundAlignment = std::min(Alignment.valueOrOne(), Align(8));
----------------
nit: `..`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69307





More information about the llvm-commits mailing list