[PATCH] D118441: Preserve aliasing info during memory intrinsics lowering

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 21 17:58:11 PST 2022


arsenm added a comment.

In D118441#3318738 <https://reviews.llvm.org/D118441#3318738>, @ebrevnov wrote:

> Do you mean it's fine to modify AMDGPULowerIntrinsics pass? If that's what you mean I don't see there is enough rational to do a non NFC change just to be able to create a test.

Yes. It's a good improvement either way, and lit tests are much more stable and maintainable than ones directly touching the APIs


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118441



More information about the llvm-commits mailing list