[PATCH] D78610: [llvm][NFC][CallSite] Remove CallSite from a few trivial locations

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 21 22:40:43 PDT 2020


craig.topper added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/MemCpyOptimizer.cpp:730
 
   // Deliberately get the source and destination with bitcasts stripped away,
   // because we'll need to do type comparisons based on the underlying type.
----------------
This comment isn't relevant to this code. There used to be some code here that modified cpySrc/cpyDest. It was removed and the comment was orphaned.

I don't think we need CB. Can't we just use C directly, its already a CallInst.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78610





More information about the llvm-commits mailing list