[PATCH] D59971: [GlobalISel] Add legalization support for non-power-2 loads and stores

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 29 15:01:15 PDT 2019


arsenm added a comment.

In D59971#1447273 <https://reviews.llvm.org/D59971#1447273>, @aemerson wrote:

> Fix alignment for smaller MMO. I couldn't use your suggested getMachineMemOperand() method because it doesn't update the alignment it seems.


Are you confusing the alignment and base alignment? I had this problem last time I touched this. getMachineMemOperand should do the right thing


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59971





More information about the llvm-commits mailing list