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

[Matt Arsenault via Phabricator via llvm-commits]

arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.
Herald added a project: All.

LGTM but a lit test would be much more maintainable



================
Comment at: llvm/unittests/Transforms/Utils/MemTransferLowering.cpp:173
         auto *TargetBB = getBasicBlockByName(F, "vector.body");
-        EXPECT_NONFATAL_FAILURE(EXPECT_NE(TargetBB, nullptr), "");
+        EXPECT_NE(TargetBB, nullptr);
         return PreservedAnalyses::all();
----------------
These operands should be swapped to get correct error messages


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118441