[PATCH] D78322: [CallSite removal][IPO] Move AbstractCallSite to its own header and change it to use CallBase instead of CallSite. NFCI

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 16 12:52:13 PDT 2020


craig.topper created this revision.
craig.topper added reviewers: jdoerfert, dblaikie, mtrofin.
Herald added a subscriber: hiraditya.
Herald added a reviewer: sstefan1.

CallSite will likely be removed soon, but AbstractCallSite serves a different purpose and won't be going away.

This patch moves it to its own header and switches it to internally store a CallBase* instead of a CallSite. The only interface changes are the removal of the getCallSite function and getCallBackUses now takes a CallBase&. These methods had only a few callers that were easy enough to update without needing a compatibility shim.


https://reviews.llvm.org/D78322

Files:
  llvm/include/llvm/IR/AbstractCallSite.h
  llvm/include/llvm/IR/CallSite.h
  llvm/include/llvm/Transforms/IPO/Attributor.h
  llvm/lib/IR/AbstractCallSite.cpp
  llvm/lib/Transforms/IPO/Attributor.cpp
  llvm/lib/Transforms/IPO/AttributorAttributes.cpp
  llvm/lib/Transforms/IPO/IPConstantPropagation.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78322.258136.patch
Type: text/x-patch
Size: 25884 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200416/b7d1f91b/attachment.bin>


More information about the llvm-commits mailing list