[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