[PATCH] D18485: [profile] Add integration test to validate PGO function names

David Li via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 25 16:23:29 PDT 2016


davidxl added inline comments.

================
Comment at: test/profile/instrprof-visibility-kinds.inc:11
@@ +10,3 @@
+
+extern void f4();
+
----------------
Is this decl needed?

================
Comment at: test/profile/instrprof-visibility.cpp:39
@@ +38,3 @@
+
+// COV-DAG: _ZN2N12f1Ev
+// COV-DAG: instrprof-visibility.cpp:_ZN12_GLOBAL__N_12f1Ev
----------------
global f2 is missing in the check?


http://reviews.llvm.org/D18485





More information about the llvm-commits mailing list