[PATCH] D102869: CodeGen: Store LLT instead of uint64_t in MachineMemOperand

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

Are we already testing the G_MEMCPY path with large (invalid) sizes?

Anyone else got any comments?



================
Comment at: llvm/lib/CodeGen/MachineOperand.cpp:1045
+  : MachineMemOperand(ptrinfo, f,
+                      s == ~UINT64_C(0) ? LLT() : LLT::scalar(8 * s), a,
+                      AAInfo, Ranges, SSID, Ordering,
----------------
Do we need to handle s > maxsize ?


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

https://reviews.llvm.org/D102869