[PATCH] D109816: [hwasan] also omit safe mem[cpy|mov|set].

Florian Mayer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 21 09:53:34 PDT 2021


fmayer added a comment.

In D109816#3012979 <https://reviews.llvm.org/D109816#3012979>, @kstoimenov wrote:

> I remember from the my C days that one of gotchas with memmove/memcpy is overlapping regions. Not sure if it is relevant the context of this change, just wanted to mention it.

Thanks! That should be okay, we look at the two parameters separately; there is even a test that has memmove to itself (`@test_in_range4`).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109816



More information about the llvm-commits mailing list