[PATCH] D91576: [MemCpyOpt] Correctly merge alias scopes during call slot optimization

Di Mo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 30 13:03:22 PST 2020


modimo added inline comments.


================
Comment at: llvm/test/Transforms/MemCpyOpt/callslot_badaa.ll:20
+; Note we're matching the full line and looking for the lack of !alias.scope here
+; CHECK:   call void @llvm.memcpy.p0i8.p0i8.i64(i8* align 8 %dst, i8* align 8 %src, i64 1, i1 false)
+  call void @llvm.lifetime.start.p0i8(i64 8, i8* nonnull %src), !noalias !3
----------------
jeroen.dobbelaere wrote:
> It would be nice if we could check that the !alias.scope really has been omitted.
added --match-full-lines in FileCheck to do just that :D


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

https://reviews.llvm.org/D91576



More information about the llvm-commits mailing list