[PATCH] D95543: [GVN] Clobber partially aliased loads.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 13 13:10:05 PDT 2021


nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

Thanks for the explanation! Probably the best solution would be the same we did in AA, which is to embed the offset in MemDepResult, then we couldn't have this kind of "out of sync" issue by design. But this also LGTM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95543



More information about the llvm-commits mailing list