[PATCH] D8940: Clang changes for indirect call target profiling
Vedant Kumar via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 8 17:30:21 PST 2016
vsk added a subscriber: vsk.
vsk added a comment.
Thanks betul! Just a minor nit.
================
Comment at: lib/CodeGen/CGCall.cpp:3550
@@ +3549,3 @@
+ DirectCallee = dyn_cast<llvm::Function> (Callee);
+ if (!DirectCallee)
+ PGO.valueProfile(Builder, llvm::IPVK_IndirectCallTarget,
----------------
Replacing this condition with `!CS.getCalledFunction()` lets us get rid of the previous conditional, IIUC.
http://reviews.llvm.org/D8940
More information about the cfe-commits
mailing list