|
@@ -15,7 +15,6 @@
|
|
|
#include "llvm/Analysis/LoopInfo.h"
|
|
|
#include "llvm/Analysis/TargetTransformInfo.h"
|
|
|
#include "llvm/Analysis/ValueTracking.h"
|
|
|
-#include "llvm/IR/CallSite.h"
|
|
|
#include "llvm/IR/DataLayout.h"
|
|
|
#include "llvm/IR/Function.h"
|
|
|
#include "llvm/Support/Debug.h"
|
|
@@ -125,14 +124,12 @@ void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB,
|
|
|
continue;
|
|
|
|
|
|
// Special handling for calls.
|
|
|
- if (isa<CallInst>(I) || isa<InvokeInst>(I)) {
|
|
|
- ImmutableCallSite CS(&I);
|
|
|
-
|
|
|
- if (const Function *F = CS.getCalledFunction()) {
|
|
|
+ if (const auto *Call = dyn_cast<CallBase>(&I)) {
|
|
|
+ if (const Function *F = Call->getCalledFunction()) {
|
|
|
// If a function is both internal and has a single use, then it is
|
|
|
// extremely likely to get inlined in the future (it was probably
|
|
|
// exposed by an interleaved devirtualization pass).
|
|
|
- if (!CS.isNoInline() && F->hasInternalLinkage() && F->hasOneUse())
|
|
|
+ if (!Call->isNoInline() && F->hasInternalLinkage() && F->hasOneUse())
|
|
|
++NumInlineCandidates;
|
|
|
|
|
|
// If this call is to function itself, then the function is recursive.
|
|
@@ -147,7 +144,7 @@ void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB,
|
|
|
} else {
|
|
|
// We don't want inline asm to count as a call - that would prevent loop
|
|
|
// unrolling. The argument setup cost is still real, though.
|
|
|
- if (!isa<InlineAsm>(CS.getCalledValue()))
|
|
|
+ if (!Call->isInlineAsm())
|
|
|
++NumCalls;
|
|
|
}
|
|
|
}
|