[PATCH] D86883: [X86] Add support for using fast short rep mov for memcpy lowering.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 31 10:47:37 PDT 2020


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86SelectionDAGInfo.cpp:198
   SDValue InFlag;
-  Chain = DAG.getCopyToReg(Chain, dl, CX, Size, InFlag);
+  Chain = DAG.getCopyToReg(
+      Chain, dl, CX,
----------------
Is this needed because we're no longer calling this with just constants we fixed the size of?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86883



More information about the llvm-commits mailing list