[PATCH] D89321: [MemLoc] Support memchr/memccpy in MemoryLocation::getForArgument
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 16 01:44:55 PDT 2020
fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.
LGTM, thanks!
================
Comment at: llvm/test/Transforms/DeadStoreElimination/MSSA/libcalls.ll:277
+
+define i8* @test_memccpy_variable_size(i8* noalias %foo, i64 %n) {
+; CHECK-LABEL: @test_memccpy_variable_size(
----------------
nit: noalias should not be needed here and the other tests, as %stack.ptr is to an alloca and LLVM should be able to determine `noalias` without the extra help.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89321/new/
https://reviews.llvm.org/D89321
More information about the llvm-commits
mailing list