[PATCH] Clang changes for indirect call target profiling

David davidxl at google.com
Tue May 19 22:11:45 PDT 2015


Does user need to specify -lpthread in the command line explicitly? This should be handled by the compiler driver when value profiling is enabled.


================
Comment at: lib/CodeGen/CGClass.cpp:1368
@@ -1367,1 +1367,3 @@
   CXXDtorType DtorType = CurGD.getDtorType();
+  Stmt *Body = Dtor->getBody();
+
----------------
This is covered by your separate patch already.

================
Comment at: lib/CodeGen/CodeGenPGO.cpp:29
@@ +28,3 @@
+  llvm::cl::desc("Disable profiling of indirect calls"),
+  llvm::cl::init(false));
+
----------------
Probably disable value profiling by default initially until it gets stablized.

An user level option is also preferred: -fprofile-values

http://reviews.llvm.org/D8940

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list