[PATCH] D22893: [sanitizer] Simplify and future-proof maybeMarkSanitizerLibraryCallNoBuiltin().

[Sunita_Marathe via llvm-commits]

Sunita_Marathe added a comment.

Hi Evgeniy,

Thank you for adding support for mempcpy to resolve the MSan fails caused by r276771.
Your change looks right to me.

memcpy() and memmov(), which are represented in the IR as intrinsics,  would also, I think, be getting marked with Attribute::NoBuiltin for the purposes of MSan testing.  They appear to be handled in a different way from the LibFunc functions.

Thanks
Sunita

Thanks
Sunita


Repository:
  rL LLVM

https://reviews.llvm.org/D22893