[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