[PATCH] D145863: [X86][NFC] Use llvm::Align for passing the alignment

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 13 02:35:50 PDT 2023


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM - but please add the PR61348 test case as well



================
Comment at: llvm/lib/Target/X86/X86SelectionDAGInfo.cpp:185
 static MVT getOptimalRepmovsType(const X86Subtarget &Subtarget,
                                  uint64_t Align) {
   assert((Align != 0) && "Align is normalized");
----------------
Probably worth converting this to Align Alignment as well?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145863



More information about the llvm-commits mailing list