[PATCH] D131577: [MemorySanitizer] Support memcpy.inline and memset.inline

Marco Elver via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 11 01:35:14 PDT 2022


melver marked 3 inline comments as done.
melver added inline comments.


================
Comment at: llvm/test/Instrumentation/AddressSanitizer/mem-intrinsics.ll:34
+  entry:
+  tail call void @llvm.memset.inline.p0i8.i64(i8* %a, i8 0, i64 100, i1 false)
+  tail call void @llvm.memcpy.inline.p0i8.p0i8.i64(i8* %a, i8* %b, i64 100, i1 false)
----------------
dvyukov wrote:
> llvm is not confused by 2 tail calls in a row?
No - the test passes, and just follows convention from above. Presumably it was done to make sure instrumentation isn't confused by tail calls? I can't say...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131577



More information about the llvm-commits mailing list