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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D102869#2784196 <https://reviews.llvm.org/D102869#2784196>, @RKSimon wrote:

> There are a number of other getMachineMemOperand methods - is the intention to add LLT variants of those as well? And deprecate the uint64_t Size variants?

Yes, faithfully preserving the memory type everywhere will take additional changes. It should only be semantically significant for G_* operations, so arguably we could keep the uint64_t versions around


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

https://reviews.llvm.org/D102869