[PATCH] D66079: [SimplifyLibCalls] Add dereferenceable bytes from known callsites [WIP]

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: lib/Transforms/Utils/SimplifyLibCalls.cpp:983
+  if (isIntrinsic)
+    return nullptr;
+
----------------
xbolva00 wrote:
> jdoerfert wrote:
> > What does the return value here mean? I'm also confused by the `Op0` which was there already.
> > 
> nullptr means that nothing was simplified.
> 
> below, for memcpy, we can just turn memcpy libcall to memcpy intrinsic.
but why is the pointer operand returned?


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

https://reviews.llvm.org/D66079