[PATCH] D86724: [BuildLibCalls] Add argmemonly to more lib calls.

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/BuildLibCalls.cpp:188
   case LibFunc_strchr:
   case LibFunc_strrchr:
     Changed |= setOnlyReadsMemory(F);
----------------
strchr too? 


================
Comment at: llvm/lib/Transforms/Utils/BuildLibCalls.cpp:318
     return Changed;
   case LibFunc_memcmp:
     Changed |= setOnlyReadsMemory(F);
----------------
Same for bcmp?


================
Comment at: llvm/lib/Transforms/Utils/BuildLibCalls.cpp:327
   case LibFunc_memchr:
   case LibFunc_memrchr:
+    Changed |= setOnlyAccessesArgMemory(F);
----------------
We miss attrs for strnlen 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86724