[PATCH] D120370: [AMDGPU] Fix combined MMO in load-store merge

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

> The thing I have written in the test might be mannered, but this is exactly what I meant for the testing purposes: align is 8, basealign is 4, offset to it is 4,

MachineMemOperand **cannot** represent that information. It only stores basealign explicitly, and it computes align on demand from basealign and offset. See MachineMemOperand::getAlign().


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

https://reviews.llvm.org/D120370