[PATCH] D21056: [profile] in-process profile merging support Part-3

Vedant Kumar via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 7 14:56:54 PDT 2016


vsk accepted this revision.
vsk added a comment.
This revision is now accepted and ready to land.

LGTM with one nit.


================
Comment at: lib/profile/InstrProfilingFile.c:34
@@ -21,2 +33,3 @@
 
+#define PROFILE_POOL_MAX_SIZE 9
 #define MAX_PID_SIZE 16
----------------
I don't think this is used anywhere, and it might be a bit cumbersome. Delete?


http://reviews.llvm.org/D21056





More information about the llvm-commits mailing list