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/



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to