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

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 12 13:35:28 PDT 2019


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





More information about the llvm-commits mailing list