[PATCH] D73964: [NFC] Introduce an API for MemOp

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 4 07:27:47 PST 2020


gchatelet marked 3 inline comments as done.
gchatelet added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/TargetLowering.h:157
   uint64_t size() const { return Size; }
   uint64_t getDstAlign() const {
+    assert(!DstAlignCanChange);
----------------
courbet wrote:
> LGTM with the understanding that this will go away and be replaced with `Align` eventually.
I've done it in this patch eventually.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73964





More information about the llvm-commits mailing list