[PATCH] D102204: [AArch64][GlobalISel] Support truncstorei8/i16 w/ combine to form truncating G_STOREs.

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 10 20:10:45 PDT 2021


aemerson added inline comments.


================
Comment at: llvm/utils/TableGen/GlobalISelEmitter.cpp:3783-3785
+  // MMO's work in bytes so we must take care of unusual types like i1
+  // don't round down.
+  return llvm::alignTo(MemTyOrNone->get().getSizeInBits(), 8);
----------------
arsenm wrote:
> Where does this actually come up? Are there really patterns using i1 memory types?
I’m not sure, this was just a refactoring of existing code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102204



More information about the llvm-commits mailing list